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

OpenRV: added integration #6185

Closed
wants to merge 8 commits into from
Closed

Conversation

kalisp
Copy link
Member

@kalisp kalisp commented Feb 1, 2024

Changelog Description

This PR is adding OpenRV - open source version of RV - image and sequence viewer for VFX and animation artists - to the list of applications.

This allows to set path to OpenpRV binaries in Applications addon and add it to Anatomy > Attributes for particular project to show up in Ayon Launcher.

Additional info

Code for addon itself is https://github.com/ynput/ayon-openrv . Doesn't contain OpenRV binaries because of licensing!

Currently implemented and working workfile workflow. There was annotation auto creator but workflow was unfinished. Unsure how the workflow should look like and what should have produced to be able to finish it. Open for comments.

Documentation for this PR here ynput/ayon-documentation#130

Testing notes:

  1. build new Applications addon via server_addon/create_ayon_addons.py
  2. deploy it to Ayon server and restart
  3. use new Applications addon in bundle
  4. set path to OpenRV in ayon+settings://applications/applications/openrv/variants/0
chrome_47wbhtvNK9 5. set `OpenRV` to `ayon+anatomy://ayon_test/attributes`

Must be added to show up in Applications and to be added in Anatomy.Attributes.
To open last opened workfile
Must be added to show up in Applications and to be added in Anatomy.Attributes.
To open last opened workfile
@kalisp kalisp requested a review from antirotor February 1, 2024 12:29
@ynbot ynbot added type: feature Larger, user affecting changes and completely new things size/XS Denotes a PR changes 0-99 lines, ignoring general files labels Feb 1, 2024
@kalisp kalisp mentioned this pull request Feb 1, 2024
15 tasks
@mkolar
Copy link
Member

mkolar commented Feb 8, 2024

I'd say there's no point merging this now with the existence of ayon-openrv

@kalisp
Copy link
Member Author

kalisp commented Feb 9, 2024

You mean ayon-core, it is still required to have it there even if the code itself is in separate repo. We still need to expose OpenRV in a list of applications.

@mkolar
Copy link
Member

mkolar commented Feb 9, 2024

You mean ayon-core, it is still required to have it there even if the code itself is in separate repo. We still need to expose OpenRV in a list of applications.

yeah let's say I fully realised that :D

@mkolar mkolar closed this Feb 9, 2024
@ynbot ynbot added this to the next-patch milestone Feb 9, 2024
@mkolar
Copy link
Member

mkolar commented Feb 9, 2024

@kalisp let's add it to ayon-core pls

@jakubjezek001 jakubjezek001 removed this from the next-patch milestone Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
port to AYON size/XS Denotes a PR changes 0-99 lines, ignoring general files target: AYON type: feature Larger, user affecting changes and completely new things
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

5 participants