Skip to content
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

Bug: Tray Publisher Editorial Simple fails on "Integrating Asset" due to no representation associated with the created shot #9

Open
2 tasks done
robin-ynput opened this issue Aug 20, 2024 · 3 comments
Assignees
Labels
type: bug Something isn't working

Comments

@robin-ynput
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior:

When publishing via Tray Publisher "Editorial Simple" to create new shots it errors with the This is not your fault error.

Here's the log:
publish-report-240820-13-44.json

Which shows error:
image

Expected Behavior:

I'm not sure about this one, are the integrating tasks relevant when simply creating a shot ?

Expected Behavior:

The current behavior properly detects and creates the sequence/shots properly, but I'm not sure what's the purpose of the Integrate Asset tasks afterwards.

Version

1.0.0

What platform you are running on?

Windows

Steps To Reproduce:

  1. Create a Folder and a Task under a project
  2. Open AYON traypublisher and select Editorial Simple
  3. Use a simple .edl and media to trigger the creation
  4. Run the publish

Are there any labels you wish to add?

  • I have added the relevant labels to the bug report.

Relevant log output:

Traceback (most recent call last):
  File "C:\Users\robin\AppData\Local\Ynput\AYON\dependency_packages\ayon_2406251801_windows.zip\dependencies\pyblish\plugin.py", line 528, in __explicit_process
    runner(*args)
  File "C:\Users\robin\OneDrive\Bureau\dev_ayon\dev\ayon-core\client\ayon_core\plugins\publish\integrate.py", line 144, in process
    filtered_repres = self.filter_representations(instance)
  File "C:\Users\robin\OneDrive\Bureau\dev_ayon\dev\ayon-core\client\ayon_core\plugins\publish\integrate.py", line 181, in filter_representations
    raise KnownPublishError(
ayon_core.pipeline.publish.publish_plugins.KnownPublishError: Instance shot has no representations to integrate

Additional context:

Addon Versions

ayon_core - 0.4.4
ayon-traypublisher - 0.2.6

@robin-ynput robin-ynput added the type: bug Something isn't working label Aug 20, 2024
@robin-ynput robin-ynput self-assigned this Aug 20, 2024
@robin-ynput
Copy link
Contributor Author

Note, from the work I've done on Resolve. It looks like a simple instance.data["integrate"] = False might be solution here.

@BigRoy
Copy link
Contributor

BigRoy commented Aug 28, 2024

Note, from the work I've done on Resolve. It looks like a simple instance.data["integrate"] = False might be solution here.

That sounds about right, yes.

I suppose you want these publish instances to solely exist for the "creation" of the folder (and/or task) entities? In that case, this sounds like the easiest approach. Yes.

Bonus points for adding a comment above the line of code that sets instance.data["integrate"] = False

For example:

        # We do not want to integrate any products for this instance. 
        # The instance is only used to generate new folder entities for
        # the shots.
        instance.data["integrate"] = False

@jakubjezek001
Copy link
Member

jakubjezek001 commented Oct 7, 2024

It seems that this had been already implemented via this PR https://github.com/ynput/ayon-traypublisher/pull/12/files#diff-1e3df0c50f4abae18adefdc7ec6d59fed745cb0bcb251882f19ceedfc597d44bR135

Could you try @robin-ynput to publish it again in latest ayon-traypublisher release 0.2.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants