Nuke: Publish and load slate frame explicitly - AY-4055

[tokejepsen]

Changelog Description

Explicitly publish representations with slateFrame data member, so loading can query whether there is slate frame.

Testing notes:

1. Publish a slate render/review from Nuke.
2. Load the render/review into Nuke.
3. Validate slate is at the Nuke workfiles first frame minus 1.

[BigRoy]

Even though I understand what it is trying to solve I feel things here are making this way more complicated than it really should be.

The assumption that a slate is only ever 1 frame seems wrong - I have also historically seen 'animated slates' that last up to a second or two, like almost a bumper with color checks, etc. I feel like the slateFrame true/false just doesn't account to that?

[tokejepsen]

The assumption that a slate is only ever 1 frame seems wrong - I have also historically seen 'animated slates' that last up to a second or two, like almost a bumper with color checks, etc. I feel like the slateFrame true/false just doesn't account to that?

Yeah, I was thinking whether to account for that, but I've personally never encountered slate longer than 1 frame.

@jakubjezek001 got any experience with this?

[jakubjezek001]

yeah, I was thinking whether to account for that, but I've personally never encountered slate longer than 1 frame.

Never had but we could be more safe if we do slateFrames as Int instead. This way in future we would be covered. But the refractory of the code for publishing plugins should happen in future on demand (those are only counting with single slate frame at the moment).

Thanks @BigRoy for mentioning this!

[tokejepsen]

Accounting for multiple slate frames now.

[jakubjezek001]

perfect! thanks @tokejepsen