-
Notifications
You must be signed in to change notification settings - Fork 850
removed italics and fixed menu formatting #6937
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
removed italics and fixed menu formatting #6937
Conversation
Hi! This comment will help you figure out which jobs to run before merging your PR. The suggestions are dynamic based on what files you have changed. HDRP Depending on the scope of your PR, you may need to run more jobs than what has been suggested. Please speak to your lead or a Graphics SDET (#devs-graphics-automation) if you are unsure. |
It appears that you made a non-draft PR! |
4. If the image does not converge over time, select the drop-down next to the effect toggle and enable Always Refresh. | ||
2. In the Inspector, select **Add Override** > **Ray Tracing** > **Path Tracing**. | ||
3. In the Inspector for the Path Tracing Volume Override, check the Enable option. If you don't see the Enable option, make sure your HDRP Project supports ray tracing. For information on setting up ray tracing in HDRP, see [getting started with ray tracing](Ray-Tracing-Getting-Started.md). This switches HDRP to path-traced rendering and you should initially see a noisy image that converges towards a clean result. | ||
4. If the image doesn't converge over time, in the Scene view toolbar, select the drop-down next to the effect toggle and enable Always Refresh. | ||
|
||
 |
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.
This screenshot needs updating.
@@ -67,23 +67,23 @@ For that reason, some parameters have no effect in path tracing, while others be | |||
|
|||
### Refraction model |
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.
Hm. I re-wrote this section and it should have been merged. I'll go investigate this.
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.
Thi page needs a lot of work but I've done most of it already :D AWe just need to figure out why it's not actually pushed... I'll approve this but be careful of merge conflicts.
The other PR is this: #6494. We have a meeting with Vic to talk about it, but it will need rewording. For now I will merge this one as it ahve small adjustement. |
* list formatting (#6934) * list formatting * Update Ray-Traced-Global-Illumination.md * removed italics and fixed list formatting (#6936) * removed italics and fixed menu formatting (#6937) * removed italics and fixed menu formatting * updated screenshot * replaced screenshot (#6941) * Updated screenshot (#6962) * note formatting and captions (#6973) * note formatting and captions * Update Post-Processing-Main.md * Initialism (#6974) * added anchor link (#6983) * paragraph restructures, formatting fixes (#6998) * fixed table subproperty (#7000) * menu path and list formatting (#7001) * Apply formatting changes Co-authored-by: emilybrown1 <88374601+emilybrown1@users.noreply.github.com> Co-authored-by: noreply@unity3d.com <noreply@unity3d.com>
* list formatting (#6934) * list formatting * Update Ray-Traced-Global-Illumination.md * removed italics and fixed list formatting (#6936) * removed italics and fixed menu formatting (#6937) * removed italics and fixed menu formatting * updated screenshot * replaced screenshot (#6941) * Updated screenshot (#6962) * note formatting and captions (#6973) * note formatting and captions * Update Post-Processing-Main.md * Initialism (#6974) * added anchor link (#6983) * paragraph restructures, formatting fixes (#6998) * fixed table subproperty (#7000) * menu path and list formatting (#7001) * Apply formatting changes Co-authored-by: emilybrown1 <88374601+emilybrown1@users.noreply.github.com> Co-authored-by: noreply@unity3d.com <noreply@unity3d.com>
Part of a quality review of HDRP docs.
Removed italics and fixed menu formatting