-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[API] Get abstract paths with a different sequence key format #783
base: master
Are you sure you want to change the base?
[API] Get abstract paths with a different sequence key format #783
Conversation
…special format value Fetch paths by ignoring abstract keys
…special format value - add tests
…ent to prevent unwanted in place edit
3e079eb
to
e7db0fe
Compare
f303f4c
to
272b21c
Compare
@jfboismenu, @manneohrstrom what do you think about this proposal ? :) |
Hi @aoblet , we're pretty bandwidth limited, as we're putting the finishing touches on Python 3 across Shotgun. This does look super interesting however, so I've logged and linked this in JIRA. We'll keep you posted. |
Jira ticket: SG-19020 |
Oh, and Manne left the team a year ago. :( |
@jfboismenu great, thanks for this quick feedback. Good luck |
f58829f
to
6b1f3dd
Compare
b550437
to
5a6ce8b
Compare
@aoblet @jfboismenu Hi guys, any update about this PR ? |
Hi @eprana @aoblet sorry that it has taken so long for someone to look at this PR. I did have a question to help me understand the use case for this functionality. I'm just wondering if you could get the same functionality by making a second template that used a key whose format spec was what you wanted instead of using and replacing the SEQ key. Or would you use this in a case where some code (eg one of the integrations perhaps) expected a SEQ key in the template but you wanted a different format spec than what is defined in the templates for SEQ? If it's the second case, would using aliases work instead (as described here https://developer.shotgridsoftware.com/82ff76f7/#example---two-fields-both-named-version-via-an-alias) ? |
Goal
Add the ability to use a different format for
SequenceKey
token withabstract_path_from_template
Use case
Find abstract paths from an houdini path template compatible for Nuke Sequence padding.
Unit tests
Tests added to check this fix:
tests/core_tests/tests_api:test_sequence_format