Skip to content

propagate launch context environment to generate project ue workers - #262

Merged
moonyuet merged 1 commit into
ynput:developfrom
dev-holobay:fix-generate-project
Feb 20, 2026
Merged

propagate launch context environment to generate project ue workers#262
moonyuet merged 1 commit into
ynput:developfrom
dev-holobay:fix-generate-project

Conversation

@nebukadhezer

Copy link
Copy Markdown
Contributor

Changelog Description

When generating a project from scratch the creation fails as the Ayon unreal init attempts to import ayon_core.

Additional review information

Need to investigate if this also needs UE_PYTHONPATH or not.

Testing notes:

  1. blank project one setup task
  2. launch unreal for the first time on a the setup task
  3. generation fails with no module found ayon_core

@nebukadhezer

Copy link
Copy Markdown
Contributor Author

would close #263

@BigRoy BigRoy linked an issue Feb 18, 2026 that may be closed by this pull request
1 task
@BigRoy
BigRoy requested review from antirotor and moonyuet February 18, 2026 09:33
@BigRoy BigRoy added type: bug Something isn't working community Issues and PRs coming from the community members labels Feb 18, 2026
@nebukadhezer

Copy link
Copy Markdown
Contributor Author

So in order for this to work, all must be provided to UE_PYTHONPATH... we could add this in front of the call, but I am hesitant if that is a good idea with my testing all Ayon related python modules are already in UE_PYTHONPATH, for my rez stuff I can easily solve that in the unreal package.py and pipe all in there then all is working.
the most comprehensive move would be to merge UE_PYTHONPATH with PYTHONPATH and make sure all is in there... but that is a choice I am leaving up to you to decide.
This will work if beforehand everyone makes sure to provide unreal relevant python modules via UE_PYTHONPATH, which to me is acceptable behaviour.

@nebukadhezer
nebukadhezer marked this pull request as ready for review February 18, 2026 17:37
@moonyuet

moonyuet commented Feb 20, 2026

Copy link
Copy Markdown
Member

the most comprehensive move would be to merge UE_PYTHONPATH with PYTHONPATH and make sure all is in there... but that is a choice I am leaving up to you to decide.

You already did this when implementing #264 as shown in https://github.com/dev-holobay/ayon-unreal/blob/ebd10367bca9414ed528825126c777265b58602a/client/ayon_unreal/hooks/pre_pyside_install.py#L222-L228. 😃

@moonyuet

Copy link
Copy Markdown
Member

I tested in 5.4. it is working. I will test 5.7 later.

@moonyuet moonyuet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested successfully in 5.7

@moonyuet
moonyuet merged commit bf10047 into ynput:develop Feb 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Issues and PRs coming from the community members type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generate Project fails in ue5.7 with ayon_core module could not be imported

3 participants