OpenPype icon indicating copy to clipboard operation
OpenPype copied to clipboard

Integration: 3DEqualizer integration

Open antirotor opened this issue 2 years ago • 4 comments

Changelog Description

This PR is adding basic integration for 3DEqualizer4 from Science-D-Vision. Integration includes:

  • Workfiles
  • Loading plates (cameras)
  • Publishing scripts to Maya and Nuke
  • Publishing of lens data

Additional info

This integration is installing PySide2 into 3DEqualizer environment as it doesn't ship with Qt support. This comes with some price - to make Qt UI work with 3DEqualizer, processEvent() is periodically called. This is not optimal and it might create some issues, like 3Dequalizer crashing or UI lags.

note More description coming

antirotor avatar Nov 03 '23 18:11 antirotor

Task linked: OP-6032 3D Equalizer integration

ynbot avatar Nov 03 '23 18:11 ynbot

@antirotor I need to communicate with you please.

I have a 3DE4 with older version and I need your help to let me integrate it. This version supports Python2.7

seifibrahim32 avatar Feb 12 '24 11:02 seifibrahim32

@antirotor I need to communicate with you please.

I have a 3DE4 with older version and I need your help to let me integrate it. This version supports Python2.7

For that, you would have to change all py3 syntax like f-strings and type hints to py2.7 syntax. From older 3de API support I can't provide much assistance unfortunately.

antirotor avatar Feb 12 '24 12:02 antirotor

@antirotor Frankly I exchanged it at the meantime with .format() as replacement to f-strings but I need to know more details for using hints for python2.7, please.

UPDATE: I understood you mean fixing interpreter hints for Py27 errors https://www.alexkorablev.com/typing-ci.html

seifibrahim32 avatar Feb 12 '24 13:02 seifibrahim32

I guess this can be merged as a base integration and I'll do port to AYON where it can be enhanced further

antirotor avatar Apr 15 '24 12:04 antirotor

I guess this can be merged as a base integration and I'll do port to AYON where it can be enhanced further

I'd actually move forward and start effort on pushing this into a separate AYON addon - and closing this here. That's at least what I've seen happen with most other open OpenPype contributions? I believe @mkolar closed many of them?

BigRoy avatar Apr 15 '24 12:04 BigRoy

Yeah, for AYON it will become separated, but since this is already done for OP and imho functioning, it would be pitty just not to merge it.

antirotor avatar Apr 15 '24 12:04 antirotor

Port to ayon is done here https://github.com/ynput/ayon-equalizer/pull/1

antirotor avatar Apr 30 '24 12:04 antirotor