Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add media reference information to inspector #62

Open
timlehr opened this issue Sep 20, 2024 · 5 comments
Open

Add media reference information to inspector #62

timlehr opened this issue Sep 20, 2024 · 5 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@timlehr
Copy link
Collaborator

timlehr commented Sep 20, 2024

Currently the inspector doesn't show any information on the media references. It would be nice if we could expose information on all media references there too (target_url, available_range, metadata, image_bounds, ..)

image

@reinecke reinecke added the good first issue Good for newcomers label Sep 20, 2024
@jspada200
Copy link

I can take this!

@jspada200
Copy link

This is probably taking me much longer then someone who works in c++ daily but was able to get a dropdown going of the media refs on a clip. Working on populating the info and dealing with reseting the selection when clicking on a new clip

2024-09-26 16 49 01

@timlehr
Copy link
Collaborator Author

timlehr commented Sep 26, 2024

Nice! This is already really cool! 👏

I think there could even be an argument made, switching that combobox value should edit the active representation on the in-memory OTIO media reference. What do you think @jminor?

@jminor
Copy link
Member

jminor commented Sep 27, 2024

This is a great start! It would be helpful if all the media reference properties were displayed also, not just the target URL.

Maybe the combobox could let you pick the active one, but all of them could be listed in detail below that. That way it is less clicking to look at them all, but you can still pick the active one easily?

If you think those enhancements would be better as a follow-up PR that's fine too. It's up to home much time you have to work on this.

@jspada200
Copy link

Thanks Guys! I only have a few hours today to work on this but I will do what I can.

@jminor jminor added the help wanted Extra attention is needed label Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants