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 to inspector #68

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

jspada200
Copy link

This is a Work-In-Progress Pull Request meant to solve: #62

Changes

Main Changes:

  • Add new section to inspector to view and modify Media References
  • Supports MissingMediaReference, ExternalReference, and ImageSequenceReference classes
  • Add attributes for:
    • Available Image Bounds
    • Available range
    • Target/Target URL base
    • Selected media reference
    • Metadata

Minor Changes:

  • Vscode is changing the import order of implot
  • Added the vscode config file to the gitignore for anyone else using vscode

Examples:

2024-09-27 11 49 53

2024-09-27 11 49 00

Copy link

linux-foundation-easycla bot commented Sep 27, 2024

CLA Not Signed

Copy link
Member

@jminor jminor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking great!

inspector.cpp Outdated Show resolved Hide resolved
inspector.cpp Show resolved Hide resolved
@jminor
Copy link
Member

jminor commented Sep 27, 2024

If you click the "Details" link next to the DCO and EasyCLA checks, there are instructions on how to get those resolved. Let us know if you get stuck or have questions about those.

Jspada200ilm and others added 23 commits October 1, 2024 10:17
…t the active media ref on dropdown change, Display additional attributes

Signed-off-by: jspadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Signed-off-by: James Spadafora <[email protected]>
Signed-off-by: jspadafora <[email protected]>
…enTimelineIO#66)

* Merge in changes from OpenTimelineIO#55 + Improve build steps formatting

Co-authored-by: David Douglas @ddouglas
Signed-off-by: Yaash Jain <[email protected]>
Signed-off-by: jspadafora <[email protected]>
…elineIO#70)

* Add LoadString function
* Add LoadUrl function
* Export LoadUrl and LoadString through emscriptem
* Update html template to add OTIO load helpers
* Document WASM load methods

Signed-off-by: Austin Witherspoon <[email protected]>
Signed-off-by: jspadafora <[email protected]>
Bumps [libs/glfw](https://github.com/glfw/glfw) from `dc557ec` to `b35641f`.
- [Release notes](https://github.com/glfw/glfw/releases)
- [Commits](glfw/glfw@dc557ec...b35641f)

---
updated-dependencies:
- dependency-name: libs/glfw
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: jspadafora <[email protected]>
Bumps [libs/opentimelineio](https://github.com/AcademySoftwareFoundation/OpenTimelineIO) from `4b3b673` to `5184c36`.
- [Release notes](https://github.com/AcademySoftwareFoundation/OpenTimelineIO/releases)
- [Commits](AcademySoftwareFoundation/OpenTimelineIO@4b3b673...5184c36)

---
updated-dependencies:
- dependency-name: libs/opentimelineio
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: jspadafora <[email protected]>
@jspada200 jspada200 force-pushed the Add-Media-Reference-to-inspector branch from bfc40d6 to 75a333d Compare October 1, 2024 17:17
@jspada200
Copy link
Author

If you click the "Details" link next to the DCO and EasyCLA checks, there are instructions on how to get those resolved. Let us know if you get stuck or have questions about those.

Thanks, I was able to figure out the DCO but my manager had me submit my name for the CLA. I am just waiting for them to let me know what needs to be done.

Thanks!

@jminor
Copy link
Member

jminor commented Oct 1, 2024

Ok, great. Let us know if there are questions about the CLA. I know some organizations can take a while to get those signed.

Copy link
Member

@jminor jminor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is approved, just waiting for CLA.

@jspada200
Copy link
Author

Just to follow up on this. Turns our that ILM does not have the CLAs setup for OTIO and needs to go through the process of getting those with Legal so it will be a bit longer this this and other PRs from ILM can be merged. Going to leave this PR open for now and stay on top of things here.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants