Skip to content

Implementing scalabity settings for planar resolutions #2059

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

Merged
merged 30 commits into from
Oct 7, 2020

Conversation

anisunity
Copy link
Contributor

@anisunity anisunity commented Sep 30, 2020

Added the scalabiity settings on the asset and the planar probe for the resolution parameter.
image
image

Testing status
Tested that the values were actually matching the one used for rendering.
Tested that the values displayed in the mono behavior matched the ones in the asset.
Tested that the High value was clamped to the atlas resolution.
HDRP tests are green at the exception of an IES test, not sure how it is related.

- Added a rough distortion frame setting and and info box on distortion materials.
- Added tests to cover the new features.
@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page)
and cancel any jobs that started on Yamato.
See the PR template for more information.
Thank you!

@anisunity
Copy link
Contributor Author

I still have to handle the migration

@anisunity anisunity force-pushed the HDRP/planar-resolution-scalability branch from 3f2674b to 8c49ddf Compare September 30, 2020 15:19
@anisunity
Copy link
Contributor Author

image
I changed the text from PlanarReflectionResolutionXXXX to ResolutionXXXX as it was taking too mcuh space.

I also supported migration.

I think this PR should be tested by QA

@sebastienlagarde sebastienlagarde requested a review from a team October 3, 2020 20:53
@sebastienlagarde sebastienlagarde changed the base branch from HDRP/staging to HDRP/rough-refrlections-option October 6, 2020 15:55
Base automatically changed from HDRP/rough-refrlections-option to HDRP/staging October 7, 2020 08:07
@sebastienlagarde
Copy link
Contributor

For QA: I test Anis PR. There is probleme with multi selection which will be fixed in a separate PR

@sebastienlagarde sebastienlagarde merged commit 0ce8be3 into HDRP/staging Oct 7, 2020
@sebastienlagarde sebastienlagarde deleted the HDRP/planar-resolution-scalability branch October 7, 2020 08:15
sebastienlagarde added a commit that referenced this pull request Oct 8, 2020
* Fix warning in ShaderPassDecal.hlsl

* Tooltip for microshadows (#2100)

* Update Alembic version to stop Alembic complaining

* Disable quad overdraw on ps4 (#2107)

* disable quad overdraw on ps4

* update doc

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>

* Enable the DXR Vertex Format test for RG (#2022)

* Add option to enable pure specular planar reflection (optimization) (#2105)

* - Added a rough refraction option on planar reflections.
- Added a rough distortion frame setting and and info box on distortion materials.
- Added tests to cover the new features.

* review changes

* removed PerceptualRoughnessToMipmapLevel_LODFlag which isn't required.

* clean more code (remove OutputSmoothPlanarReflection)

* missing one update

* removed rough distortion

* remove distortion test scene

* Update DistortionUIBlock.cs

* clean distortion

* update documentation

* address doc feedback

* Update Upgrading-from-2020.1-to-2020.2.md

* Update Upgrading-from-2020.1-to-2020.2.md

* update test 2220

* update smoothness test

* Create 2220_SmoothPlanarReflection.png

* update reference screenshots

Co-authored-by: Anis <anis@unity3d.com>

* Implementing scalabity settings for planar resolutions (#2059)

* - Added a rough refraction option on planar reflections.
- Added a rough distortion frame setting and and info box on distortion materials.
- Added tests to cover the new features.

* review changes

* Added scalability settings for the planar reflection resolution.

* support migration.

* renamed the resolution parameters for the planar reflections

* removed PerceptualRoughnessToMipmapLevel_LODFlag which isn't required.

* clean more code (remove OutputSmoothPlanarReflection)

* missing one update

* removed rough distortion

* remove distortion test scene

* Update DistortionUIBlock.cs

* clean distortion

* update documentation

* address doc feedback

* Update Upgrading-from-2020.1-to-2020.2.md

* Update Upgrading-from-2020.1-to-2020.2.md

* update test 2220

* Revert "renamed the resolution parameters for the planar reflections"

This reverts commit 753af88.

* cleanup migration code

* update smoothness test

* Create 2220_SmoothPlanarReflection.png

* some cleanup

* Revert "Revert "renamed the resolution parameters for the planar reflections""

This reverts commit fbf3572.

* update doc

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>

* Remove useless SampleSkyTexture function

* Fix errors when resizing the compositor's output and when removing/adding the compositor (#2118)

# Conflicts:
#	com.unity.render-pipelines.high-definition/CHANGELOG.md

* Remove BLEND keywords and replace with constant branch  (#1884)

* Removing blend keyword.

* Changelog

* Update DecalProperties.hlsl

* revert change in AxF shader

* fix compile issue with ordering when using VT

* update comment

* Update Upgrading-from-2020.1-to-2020.2.md

* update comment + optimized code version (in case of bad compiler)

* update comment revert (wrong) optimization

* remove unused multicopmile in test

* candidate fix for VFX

* Revert "candidate fix for VFX"

This reverts commit e3183f0.

* Another tentative fix

* Try differently

* Fix compile issue.

* Another vfx fix attempt.

* Latest flavour of fix.

* Remove old defines

* Fix hybrid

* Update includes in ray tracing

* Update Upgrading-from-2020.1-to-2020.2.md

Co-authored-by: Sebastien Lagarde <sebastien@unity3d.com>

* Mixed cached shadow maps (Cached static + dynamic rendered every frame) (#1559)

* UI for mixed cached shadows

* Baby steps (# 1)

* Baby steps (# 2)

* Extrapolate the update request data into a function

* Commit before blit code

* Need to switch branch

* Need to switch branch

* Working as long the light does not move.

* Fixed area light case

* Make it render graph ready

* Make sure we correctly update the freshly updated cached shadows

* Raw update upon movement option (Missing API for threshold)

* Threshold API

* Add unity binary tag

* New docs

* Shader name change

* Update docs

* Fix bad merge

* Update Shadows-in-HDRP.md

* For now disable the directional light case.

* Disable directional mixed.

* Make it disabled under define

* Remove docs

* Implement rendergraph version

* Remove superflous blit

* Tentative fix for yamato.

Co-authored-by: JordanL8 <lewis.jordan@hotmail.co.uk>

* update meta file with wrong information

Co-authored-by: Adrien de Tocqueville <adrien.tocqueville@unity3d.com>
Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: Anis <anis@unity3d.com>
Co-authored-by: Pavlos Mavridis <pavlos.mavridis@unity3d.com>
Co-authored-by: FrancescoC-unity <43168857+FrancescoC-unity@users.noreply.github.com>
Co-authored-by: JordanL8 <lewis.jordan@hotmail.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants