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

Farm rendering changes #165

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

antirotor
Copy link
Member

@antirotor antirotor commented Nov 19, 2024

Changelog Description

Changes to simplify logic for farm rendering.

Additional review information

If you want to have reviews on farm, you need to add unreal with .* regex here: ayon+settings://deadline/publish/ProcessSubmittedJobOnFarm/aov_filter

Fixes:

  • refactored farm rendering flow
  • review set on instance
  • label in publisher showing folder path

Testing notes:

  • create new rendering instance.
  • set the render target for Local Rendering
  • (render first the images)
  • publish - it should publish as expected
  • switch the rendering instance to Farm Rendering
  • publish - you should have job on farm

@antirotor antirotor linked an issue Nov 19, 2024 that may be closed by this pull request
1 task
@antirotor antirotor self-assigned this Nov 20, 2024
@antirotor antirotor added the type: bug Something isn't working label Nov 20, 2024
@antirotor antirotor marked this pull request as ready for review November 20, 2024 16:29
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.

The first local render test went smoothly and all good.

But once trying to use farm render it gives me following error in the Publisher:

Traceback (most recent call last):
  File "C:\Users\lbate\AppData\Local\Ynput\AYON\dependency_packages\ayon_2406251801_windows.zip\dependencies\pyblish\plugin.py", line 528, in __explicit_process
    runner(*args)
  File "C:\Users\lbate\AppData\Local\Ynput\AYON\addons\deadline_0.4.0\ayon_deadline\plugins\publish\global\submit_publish_job.py", line 393, in process
    representations = prepare_representations(
                      ^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Work\REPO\ayon-core\client\ayon_core\pipeline\farm\pyblish_functions.py", line 336, in prepare_representations
    frames_to_render = _get_real_frames_to_render(frames_to_render)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Work\REPO\ayon-core\client\ayon_core\pipeline\farm\pyblish_functions.py", line 478, in _get_real_frames_to_render
    for frame in frames.split(","):
                 ^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'split'

@moonyuet
Copy link
Member

moonyuet commented Nov 21, 2024

I try to render out and submit to farm, it is rendered successfully.
image

image

@moonyuet
Copy link
Member

moonyuet commented Nov 21, 2024

The first local render test went smoothly and all good.

But once trying to use farm render it gives me following error in the Publisher:

Traceback (most recent call last):
  File "C:\Users\lbate\AppData\Local\Ynput\AYON\dependency_packages\ayon_2406251801_windows.zip\dependencies\pyblish\plugin.py", line 528, in __explicit_process
    runner(*args)
  File "C:\Users\lbate\AppData\Local\Ynput\AYON\addons\deadline_0.4.0\ayon_deadline\plugins\publish\global\submit_publish_job.py", line 393, in process
    representations = prepare_representations(
                      ^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Work\REPO\ayon-core\client\ayon_core\pipeline\farm\pyblish_functions.py", line 336, in prepare_representations
    frames_to_render = _get_real_frames_to_render(frames_to_render)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Work\REPO\ayon-core\client\ayon_core\pipeline\farm\pyblish_functions.py", line 478, in _get_real_frames_to_render
    for frame in frames.split(","):
                 ^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'split'

you need to test with the PR ynput/ayon-deadline#49 along with the ayon-core mentioned.

@LiborBatek
Copy link
Member

@moonyuet I will do! ...it wasnt mentioned within the testing steps...so thanks for letting me know!

@LiborBatek
Copy link
Member

Im getting this error when trying to get Layout into UE... I know that its maybe not directly related but I cant do any testing without it actually. The weird thing is, that it even happens in ofical addon versions atm

image

@moonyuet
Copy link
Member

Im getting this error when trying to get Layout into UE... I know that its maybe not directly related but I cant do any testing without it actually. The weird thing is, that it even happens in ofical addon versions atm

image

i dont think it's related to this PR though. Which unreal version are you testing?

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

Successfully merging this pull request may close these issues.

AY-7175_Rendering: No review is generated on farm
3 participants