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

Fusion basic integration #452

Merged
merged 28 commits into from
Oct 6, 2020
Merged

Conversation

jakubjezek001
Copy link
Member

@jakubjezek001 jakubjezek001 commented Aug 18, 2020

Adding basic launching methods and updating fusion integration from latest Colorbleed config repository
https://github.com/Colorbleed/colorbleed-config

Basic functionality:

  • launchers in pype
  • integrating pype menu into fusion scripts
  • merging Colorbleed-config's fusion development
  • rendering locally or deadline
  • creating, loading, inventory

Tested on Fusion 16 (licence) and Fusion 9 (free)

Testing this PR
A. Installing Fusion's own python 3.6 interpreter into path defined in pype-config/environments/fusion.json:PYTHON36 . We are using Python 3.6.2

B. Installing PySide2 into python 3.6 for correct gui work:

As Pype is using PySide2 inside installation python 3.6 as it is explained above. Follow this step-by-step:
C:\Python\Python36\python.exe -m pip install PySide2

dependencies:
ynput/pype-config#76

@jakubjezek001 jakubjezek001 requested a review from a team August 18, 2020 14:51
@jakubjezek001 jakubjezek001 self-assigned this Aug 18, 2020
@jakubjezek001
Copy link
Member Author

Hi @BigRoy could you share with me what are the basic environment variables you are using in studio. Also I am not sure if you are actually using Fusionless any more as I don't see any imports of the code in colorbleed-config. Thank you.

@BigRoy
Copy link
Collaborator

BigRoy commented Aug 19, 2020

We did indeed drop the use for Fusionless in core itself. I might still have it scattered throughout code of our config, not sure. :)

Regarding environment, I'd have to look that up but unfortunately I am out of office for a few weeks.

Also make sure the acre branch on colorbleed repo isn't more ahead with Fusion things. That's what we are using.

@jakubjezek001
Copy link
Member Author

Also make sure the acre branch on colorbleed repo isn't more ahead with Fusion things. That's what we are using.

Good point, thank you. No worries about environment vars. We will be copying scripts into appropriate folder with pre-launch hook.

@jakubjezek001 jakubjezek001 marked this pull request as ready for review August 21, 2020 12:58
@jakubjezek001 jakubjezek001 added the type: enhancement Enhancements to existing functionality label Aug 21, 2020
@mkolar mkolar force-pushed the feature/451-Fusion_basic_integration branch from b15d621 to bcb7201 Compare October 6, 2020 14:06
@mkolar
Copy link
Member

mkolar commented Oct 6, 2020

Works. However and important point needs to be documented.

It requires python 3.6.x installation and environment variable PYTHON36 pointed to it's location. And every user must have PySide2 installed in that python36 for this to work. Same problem as with Blender and Resolve.

@mkolar mkolar merged commit 1cf97a7 into 2.x/develop Oct 6, 2020
@BigRoy
Copy link
Collaborator

BigRoy commented Oct 6, 2020

Are you sure the Python install needs to be local + that PySide2 needs to be local? At the very least I'm sure we are running PySide2 on the server for Fusion - and I believe Python too (but not sure...)

It does have the side-effect that it triggers the Qt UIs relatively slow as it loads Python+PySide2 over the network.

@mkolar
Copy link
Member

mkolar commented Oct 7, 2020

@BigRoy you are right, they don't have to be local necessarily, however the speed penalty is quite substantial in certain scenarios we encountered. Also, we are generally getting ready with pype 3.0 to run fully locally for each user....Remote workflows rule the world nowadays :)

@mkolar mkolar deleted the feature/451-Fusion_basic_integration branch October 13, 2020 21:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Fusion type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fusion basic integration
4 participants