Skip to content

Conversation

lluisCM
Copy link
Contributor

@lluisCM lluisCM commented Jul 12, 2024

Resolves :

  • I have added automatic tests where applicable.
  • The PR contains a description of what has been changed.
  • The description contains manual test instructions.
  • The PR contains update to the release notes.
  • The PR contains update to the documentation.

This PR has been tested on :

  • Windows.
  • MacOs.
  • Linux.

Overview

Purpose: Premiere implementation of the Loader.

Scope: framework-premier

Implementation Details

Approach:

Reasoning:

Changes:

Trade-offs:

Testing

Tests Added:

Manual Testing:

Testing Environment:

Notes for Reviewers

Focus Areas:

Dependencies:

Known Issues:

@lluisCM lluisCM requested a review from a team as a code owner July 12, 2024 09:53
from ftrack_framework_core.plugin import BasePlugin
from ftrack_framework_core.exceptions.plugin import PluginExecutionError

from ftrack_utils.paths import check_image_sequence
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Use clique instead

)
if store.get('is_sequence'):
# Compile members
sequence_metadata = check_image_sequence(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Use clique instead

@lluisCM lluisCM added invalid This doesn't seem right connect3 PR for connect 3 experimental Experimental branches and removed invalid This doesn't seem right labels Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connect3 PR for connect 3 experimental Experimental branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant