Skip to content

Conversation

@srikanthsreeram
Copy link
Collaborator

No description provided.

Copy link

@aparna-d2 aparna-d2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we create all new templates or just allow mp4 as an alternate extension to mov?

@srikanthsreeram
Copy link
Collaborator Author

Garvit suggested not to add as an alternate extension to mov.
Since only editors are making mp4, thought to set up for nuke studio and ingestion process.

@aparna-d2
Copy link

Yes, and only texturing artists export textures, but we don't really restrict the item type to mari only. We could make the same argument about any item type, really. Is there a reason that otherwise, mp4 should not be recognised?
I don't see the harm in adding it as an alternate extension, since the rest of the template is the same. I'm just looking to limit the number of templates we are creating. Ultimately, there is no harm either way. I think this will make maintenance easy and there's no reason not to do it.
If there is a reason, let me know.

@garvitverma
Copy link
Collaborator

The reason was, I wasn't sure if any of our tools that create MOVs would then also expect to work with multiple MOV format exposed in a new key like we do for images and geometry caches. This was supposed to wait until that is figured out and the code made compatible by removing any hard-coding in tools that might be generating movs.

Till then I said restrict these templates to the show and enable the editors, before we modify existing templates that are in use. This wasn't supposed to go in the package yet.

@srikanthsreeram
Copy link
Collaborator Author

sorry, Garvit.
Thought that editors will be working on multiple shows if we enable one show they might ask for other shows as well.

Checked with Prashanth, he requires mp4 publish for HIRANYA now.
I will set up templates for HIRANYA now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants