Update Rendering section in unreal docs#355
Conversation
MustafaJafar
added
type: enhancement
LiborBatek
left a comment
LiborBatek
left a comment
There was a problem hiding this comment.
LGTM
PS. we will need to also update the user section in /docs/addon_unreal_artist#farm-rendering as its outdated. maybe just via link to the admin section??
OMG, this is way obsolete more than I thought 😅 |
|
@LiborBatek |
| AYON requires `Render Queue` and `Render Configuration` uassets to be physically present in the Unreal project. | ||
|
|
||
| By default they are expected at these paths: | ||
| By default, they are expected at these locations. To customize, see the note **Render Setup Settings** below: |
There was a problem hiding this comment.
| By default, they are expected at these locations. To customize, see the note **Render Setup Settings** below: | |
| By default, they are expected in the following locations: |
and you should move that second sentence of customizing to the Note itself instead...
There was a problem hiding this comment.
The second sentence is added to refer to the note below in case they want to customize it.
Does it look confusing ?
There was a problem hiding this comment.
to me its a bit redundant as one can read it directly then within a Note itself imho... or at least should be placed just below the file paths examples for renderQueue and Config files...
There was a problem hiding this comment.
It could be, I think it worth it mentioning that this is configurable.
Co-authored-by: Libor Batek <112623825+LiborBatek@users.noreply.github.com>
|
any further update on this PR or it's good to merge? |
| Unreal integration also supports rendering on Deadline. The process for creating a render instance is the same as mentioned above, but you need to select `Farm Rendering` in `Render Target`. | ||
|
|
||
| Deadline rendering process expects physically existing Render Queue and Render Settings uassets in the Unreal project. | ||
| AYON requires `Render Queue` and `Render Configuration` uassets to be physically present in the Unreal project. |
There was a problem hiding this comment.
Actually these config files are mandatory also when local rendering happens...I guess we need to add it also at the begining of "Render Section" in the user docs or just move it there (single mentioning for both local/farm rendering)...
|
In the Artists Rendering section in we should adjust it to something like: |
Just for reference, The docs are being migrated to featurebase. even if this PR is merged. I'm including the information in this PR to feature base and we can continue polishing it there. |
|
I've included the changes in the PR to featurebase. |
|
Changes in this PR were moved to our new help center, check them here https://help.ayon.app/collections/2313572-unreal |
3 participants
Changelog Description
resolve #354
Testing notes: