ynput / OpenPype

Main OpenPype repository and AYON openpype addon codebase
https://openpype.io
MIT License
280 stars 128 forks source link

OpenRV: added integration #6185

Closed kalisp closed 4 months ago

kalisp commented 5 months ago

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 https://github.com/ynput/ayon-documentation/pull/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
mkolar commented 5 months ago

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

kalisp commented 5 months ago

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 commented 4 months ago

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 commented 4 months ago

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