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

Remove non-core adapters #1725

Merged
merged 19 commits into from
Jun 12, 2024
Merged

Remove non-core adapters #1725

merged 19 commits into from
Jun 12, 2024

Conversation

reinecke
Copy link
Collaborator

If there is an associated issue, link it in the form:

This PR covers the major work of #1386.

Use one line for each Issue. This allows auto-closing the related issue when the fix is merged.

Summarize your change.

This PR applies core repo cleanup after creating separate repos for adapters in the OpenTimelineIO GitHub org.

  • Removes contrib adapters and tree
  • Removes all but the "native" OTIO adapters from py-opentimelineio/opentimelineio/adapters
  • Updates to plugin system and documentation to support adapter removal

This PR is the result of contributions from the broad community including:

@reinecke reinecke added this to the Public Beta 17 milestone Apr 11, 2024
@reinecke reinecke added python Pull requests that update Python code Adapters Relating to the adapters that live in the separate repos labels Apr 11, 2024
.github/workflows/docs.yml Outdated Show resolved Hide resolved
.github/workflows/python-package.yml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/tutorials/adapters.md Outdated Show resolved Hide resolved
docs/tutorials/otio-plugins.md Show resolved Hide resolved
docs/tutorials/quickstart.md Outdated Show resolved Hide resolved
setup.py Show resolved Hide resolved
@reinecke
Copy link
Collaborator Author

Build failing due to scikit-build/cmake-python-distributions/pull/490

@reinecke reinecke force-pushed the extract_adapters_post_beta_16 branch 2 times, most recently from fbe098e to 7f09945 Compare April 12, 2024 05:22
@reinecke reinecke force-pushed the extract_adapters_post_beta_16 branch 3 times, most recently from 1e8c26b to 9d33bd3 Compare June 12, 2024 22:24
Copy link
Contributor

@apetrynet apetrynet left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Thanks @reinecke !

apetrynet and others added 17 commits June 12, 2024 16:17
* add "extract_adapters" to CI triggers
* use "otiod" as override example

Signed-off-by: apetrynet <flehnerheener@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* Removed the cmx_3600 adapter
* Removed sample data only used by the cmx_3600 adapter
* Add "extract_adapters" to CI triggers
* otioz test called for an edl we removed. Replaced with an otio file
* Converted screening_example.edl to screening_example.otio and used it in tests
* Removed other adapter suffixes from plugin tests as they belong to adapters soon to be extracted
* Autogenerated docs for CMX3600 removed

Signed-off-by: apetrynet <flehnerheener@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
…1348)

* Remove AAF adapter
* Add Note about the AAF adapter being moved

Signed-off-by: Mark Reid <mindmark@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* removing the fcp adapter and it's test files
* replaced premiere_example.xml with premiere_example.otio to pass console tests.
* updated auto generated docs

Signed-off-by: apetrynet <flehnerheener@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* removed svg adapter and related test files
* updated auto documentation

Signed-off-by: apetrynet <flehnerheener@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* Remove maya adapter related files
* Update docs

Signed-off-by: rosborne132 <ozborne132@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* remove fcp x xml adapter files
* update docs

Signed-off-by: rosborne132 <ozborne132@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
Signed-off-by: rosborne132 <ozborne132@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* remove ale adapter related files

Signed-off-by: rosborne132 <ozborne132@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* extract burnin files
* remove test

Signed-off-by: rosborne132 <ozborne132@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* extract xges adapter files

Signed-off-by: rosborne132 <ozborne132@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
* Removed the "contrib" directory tree and all references to the contrib adapters.
* Plugin system, setup and auto doc scripts no longer rely on contrib files.
* Also removed some entries on other adapters left behind in the adapters.md file.
* add README_contrib.md to the manifest

Signed-off-by: apetrynet <flehnerheener@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
Signed-off-by: Tim Lehr <tim.lehr@disneyanimation.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
…tions (#1704)

Signed-off-by: Tim Lehr <tim.lehr@disneyanimation.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
Co-authored-by: Daniel Flehner Heen <flehnerheener@gmail.com>
Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
…ndscape

Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
@reinecke reinecke force-pushed the extract_adapters_post_beta_16 branch from 9d33bd3 to b052e57 Compare June 12, 2024 23:17
@reinecke reinecke merged commit e10fdd2 into main Jun 12, 2024
57 checks passed
@reinecke reinecke deleted the extract_adapters_post_beta_16 branch June 12, 2024 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Adapters Relating to the adapters that live in the separate repos python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants