Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Synchronize ftrack hierarchical and shot attributes #716

Merged

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Nov 13, 2020

Push attributes event handler now works slightly different. Pushing attribute values from pype's hierachical to not hierarchical for ftrack expressions has more features in.

Changes

  • there is no more mapping From attribute key -> To attribute key but only defined attribute keys that will be updated
    • attribute values are synchronized both ways hierarchical <-> not hierarchical
  • by default it works only for keys frameStart and frameEnd and entity type Shot but it should work for all of them (just change attributes)

How to test

  • create frameStart custom attribute for entity type Shot (must have same attributes: non-decimal number)
  • run event server
  • change value in hierarchical attribute on Shot entity -> check if was value was also changed in the new attribute
  • change value in the new attribute on Shot entity -> check if was value was also changed in hierarchical attribute

Ftrack default attributes fend and fstart can be renamed.

@iLLiCiTiT iLLiCiTiT self-assigned this Nov 13, 2020
@iLLiCiTiT iLLiCiTiT requested a review from a team November 13, 2020 15:27
@mkolar mkolar added this to the 2.14 milestone Nov 17, 2020
@mkolar mkolar added the type: enhancement Enhancements to existing functionality label Nov 17, 2020
@mkolar mkolar changed the title Push frames ftrack event handle works for all same name attributes Synchronize ftrack hierarchical and shot attributes Nov 17, 2020
@mkolar mkolar merged commit 537da30 into 2.x/develop Nov 17, 2020
@mkolar mkolar added the 2x-dev label Nov 17, 2020
@mkolar mkolar deleted the feature/push_frames_works_for_all_same_name_attributes branch December 1, 2020 15:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants