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

TrayPublisher: set default frame values to sequential data #5530

Merged
merged 4 commits into from
Sep 7, 2023

Conversation

jakubjezek001
Copy link
Member

Changelog Description

We are inheriting default frame handles and fps data either from project or setting them to 0. This is just for case a production will decide not to injest the sequential representations with asset based metadata.

Testing notes:

  1. Open TrayPublisher on a context
  2. drop some imagesequence data to Plate family.
  3. Disable instance attribute for Collect Sequence Frame Data
  4. Publish and see that all had been published as expected.
  5. look into Loader and find the published product and see that the frame-range defined on the version is the same as the original frame-range and is not the same as it is defined on asset parent.

@ynbot
Copy link
Contributor

ynbot commented Aug 29, 2023

Task linked: OP-3949 Traypublisher: plate publish

@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working labels Aug 29, 2023
@jakubjezek001 jakubjezek001 requested review from 64qam, iLLiCiTiT and jrsndl and removed request for jrsndl August 29, 2023 08:43
Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

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

When Collect sequence frame data not activated it correctly doesnt add any handle start and handle end when Publish via Tray Publisher.

Works fine.

@jakubjezek001 jakubjezek001 assigned BigRoy and unassigned LiborBatek Sep 4, 2023
@jakubjezek001 jakubjezek001 merged commit fc662b3 into develop Sep 7, 2023
7 checks passed
@jakubjezek001 jakubjezek001 deleted the bugfix/OP-3949_Traypublisher-plate-publish branch September 7, 2023 12:52
@ynbot ynbot added this to the next-patch milestone Sep 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: TrayPublisher size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants