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

Resolve: update to new publisher #3

Closed

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Jul 2, 2024

PR moved from ynput/ayon-core#157

Note

Description is work in progress

Changelog Description

Resolve with adapted latest publisher tool.

Additional info

Adaptation of ynput/OpenPype#5709 (thanks @BigRoy for the kick-off!)

TODO:

  • - @BigRoy reported that there is auto timeline created which should not
  • - @BigRoy suggested warning message if there is not anything to be used for creation
  • - @BigRoy suggested to add plugin info text under ?
  • - Spliting product types into:
    - primary: shot - instance with hierarchy, future context data and attributes placeholder
    - secondary: plate, audio, review - instance with media
  • - Hierarchy and context data should be linked from Shot instance for all secondary product types Traypublisher (TP)
  • - Each secondary product type which will be created from a clip should have own creator class - like in TP
  • - OTIO should be using native format rather then our implementation - this way we will be ready for Tony's (@tweak-wtf) retiming.

Testing notes:

To be created

jakubjezek001 and others added 7 commits July 2, 2024 15:11
- Removed the function for loading stylesheets in the menu.
- Refactored plugin creation logic to use new API methods and data structures.
- Refactored function names and descriptions for clarity
- Added new function to export timeline otio files
- Improved temporary directory handling for otio files
Adjusted get_otio_temp_dir to set timeline if not provided, throwing error if none found.
@ynbot
Copy link
Contributor

ynbot commented Jul 2, 2024

Task linked: OP-8066 Resolve: update to new publisher

@mkolar mkolar added the type: enhancement Improvement of existing functionality or minor addition label Aug 7, 2024
@mkolar mkolar requested a review from BigRoy August 7, 2024 18:44
@jakubjezek001 jakubjezek001 self-requested a review August 14, 2024 08:43
@jakubjezek001 jakubjezek001 marked this pull request as draft August 14, 2024 08:44
@dee-ynput
Copy link

Please note the potential overlaps with 🚀 Extended Publisher Epic.

@jakubjezek001
Copy link
Member

Closing this PR in favor of #15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants