Nuke: removing redundant Ftrack asset when farm publishing

[jakubjezek001]

Brief description

No need to publish sequence with review if baked profiles are used.

Description

This was happenning due wild card regex match but since we are adding bakingNukeScripts atribute to instance data, it is quite clear we don't need to review also .exr image sequence.

Additional info

We had also not very nice way of Ftrack asset naming in case multiple reviewable representations were processed. I had hopefully preserved backward compatibility for single reviewable, but it needs to be tested. The feature was made optional and can be activated in settings project_settings/ftrack/publish/IntegrateFtrackInstance/nicer_asset_name

Testing notes:

Ideal test scenarios are following:

• activate project_settings/ftrack/publish/IntegrateFtrackInstance/nicer_asset_name on your project
• nuke single baking preset - ftrack component asset name should be in old way (subset name)
• nuke multiple baking presets - asset name should be in new way (each component {subset}_{repre.name})
  

Todo:

• I have noticed we are also generating correct thumbnails for each reviewable, but those are not used anywhere. Anyway IntegrateFttrackInstances is not supporting multiple Thumbnails so this could be also improved.
  
  Problem is that each of the representations (noLut, withLut) is having differently looking image and this will be great to display as thumbnail.

[ynbot]

Task linked: OP-2988 Nuke: redundant review repre when farm publishing