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

Hiero: Initial frame publish support #3106

Merged

Conversation

jakubjezek001
Copy link
Member

@jakubjezek001 jakubjezek001 commented Apr 27, 2022

Brief description

Frame family publishing (convert form 2.x)

Description

With the "frame" family tags, you can publish still images from the edit by tagging frames in the sequence.

Original PR:

#1172

Testing notes:

  1. start your testing project in hiero
  2. Look to Tags and see FrameMain is there
  3. drop it on timeline where clips are available
  4. Publish

@jakubjezek001 jakubjezek001 self-assigned this Apr 27, 2022
@ynbot
Copy link
Contributor

ynbot commented Apr 27, 2022

@jakubjezek001 jakubjezek001 added host: Hiero type: feature Larger, user affecting changes and completely new things labels Apr 27, 2022
Copy link
Member

@tokejepsen tokejepsen left a comment

Choose a reason for hiding this comment

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

Looks good to me. Dont know what the deletions and the hierarchy collection modification are about?


# filter out all unepropriate instances
if not instance.data["publish"]:
continue

# exclude other families then self.families with intersection
if not set(self.families).intersection([family]):
if not set(self.families).intersection(set(families + [family])):
Copy link
Member Author

Choose a reason for hiding this comment

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

@tokejepsen this is just an improvement of the code. The test was only performed on main family and not on additional families.

# "marginIn": 1,
# "marginOut": 1
# }
# },
Copy link
Member Author

Choose a reason for hiding this comment

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

@tokejepsen this was just an old commented code which could be ditched..

@jakubjezek001 jakubjezek001 merged commit 53b8c63 into develop Apr 29, 2022
@jakubjezek001 jakubjezek001 deleted the feature/OP-2831_Hiero-Initial-frame-publish-support branch April 29, 2022 08:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Hiero type: feature Larger, user affecting changes and completely new things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants