search

ynput / OpenPype

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

Nuke: Update camera node type to Camera3 as Camera2 is deprecated in Nuke 15 #6196

Closed fabiaserra closed 5 months ago

fabiaserra commented 5 months ago

Testing notes:

  1. Load a camera in OP and try render it out in Nuke 15
jakubjezek001 commented 5 months ago

@moonyuet didn't we do this fix already?

moonyuet commented 5 months ago

@moonyuet didn't we do this fix already?

we fixed the one in the creator but not the loader

jakubjezek001 commented 5 months ago

here is related PR I am talking about https://github.com/ynput/OpenPype/pull/6118

Perhaps it would be better to implement it it the way @fabiaserra

moonyuet commented 5 months ago

here is related PR I am talking about #6118

Perhaps it would be better to implement it it the way @fabiaserra

Agreed. It is safer to implement the function of camera creations which supports different nuke versions.

fabiaserra commented 5 months ago

And are you sure we want Camera4 instead of Camera3? My artists said they wanted Camera3

mkolar commented 5 months ago

Because we're splitting OpenPype into ayon-core and individual host addons, this PR would have to be re-created to target one of those.

We're closing it down, but we'll he happy for a new PR to ynput/ayon-core or the host addon repository once it's up.