-
Notifications
You must be signed in to change notification settings - Fork 14
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
fix: Resolve menu.py not found error when running pip install deadline-cloud-for-nuke -t <folder> #165
Conversation
…ne-cloud-for-nuke -t <folder>` Signed-off-by: Cherie-Chen <[email protected]>
Signed-off-by: Cherie-Chen <[email protected]>
Signed-off-by: Haejung Choi <[email protected]>
3c45457
to
171150a
Compare
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
$ export NUKE_PATH=${NUKE_PATH}:<target-directory>/deadline/nuke_submitter | ||
``` | ||
|
||
After installation and within the same terminal, run `Nuke<version>` executable. The Nuke submitter should now be available in the AWS Deadline menu. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'd be helpful to give an example command here, especially for Macs where it's less obvious.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will add this on the separate PR! 🙏 Thanks for the suggestion.
To install the submitter manually, you can use pip. | ||
|
||
```sh | ||
$ pip install deadline-cloud-for-nuke -t target-directory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: It'd be slightly better to choose a real target directory like ~/deadline-cloud-for-nuke
so that users can copy-paste the commands. Then you could also note that users can choose a different directory if they want.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, we'd then also remove the shell indicator $
(for lack of knowing the term) to allow the copy/paste.
…stall deadline-cloud-for-nuke -t <folder> (#165)" (#170) This reverts commit 82b7eb2. Signed-off-by: Steven Banaszak <[email protected]>
What was the problem/requirement? (What/Why)
Addressing user feedback on receiving doesn't have menu.py error while installing deadline-cloud-for-nuke using pip install deadline-cloud-for-nuke -t command in MacOS
This PR is originated from PR-152 by Cherie. This commit is addressing:
pip install deadline-cloud-for-nuke
What was the solution? (How)
What is the impact of this change?
deadline-cloud-for-nuke
How was this change tested?
Did you run the "Job Bundle Output Tests"? If not, why not? If so, paste the test results here.
Was this change documented?
Yes - both development.md and readme.md are updated
Is this a breaking change?
No
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.