Skip to content

Hdrp/fix 1299233 ies resize #3243

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 29 commits into from
Mar 1, 2021
Merged

Conversation

skhiat
Copy link
Contributor

@skhiat skhiat commented Jan 22, 2021

recreate #3094

Fix:
https://fogbugz.unity3d.com/f/cases/1299233/ (cf. Video)
image
Repro bug:
To see Atlas Display Cookie Atlas (make sure Clear Cookie Atlas is NOT checked)

  1. Change IES Size to 64
  2. Change IES Size to 512
    Notice when clear cookie atlas we don't have the issue.

This PR fix this issue.
Make sure that don't break the others features of the Atlas (for example Animated RenderTexture).
NZueNdZniI

JulienIgnace-Unity and others added 27 commits January 8, 2021 14:01
Co-authored-by: Sebastien Lagarde <sebastien@unity3d.com>
* Not checking NdotL since it's not really valid for area lights (We have multiple valid light directions, not one)

* Changelog

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

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
* Fixed ShaderGraph decal draw order

* Updated changelog

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
* Fixed access to invalid Contexts references after exiting playmode.

* Fixed comparison gizmo after playmode.

* Fixes from PR feedback
… conserving specular color option for the specular input parametrization (similar to case 1257050) (#3060)
* adding a shadow mask test

* Update reference images
…ity) to improve visual quality. (#3020)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
…er (case 1303504). (#3027)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
…). (#3029)

* - Changed the warning message for ray traced area shadows (case 1303410).

* Adds approximation information about ray-traced area shadows

Co-authored-by: Lewis Jordan <lewis.jordan@hotmail.co.uk>
Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
…ale rtao > 1.25 with a 25cm falloff interval. (#3023)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
* Fixed Render Graph immediate mode. (#3033)

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

* Fix issue with shadow mask and area lights (#3019)

* Not checking NdotL since it's not really valid for area lights (We have multiple valid light directions, not one)

* Changelog

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

* Fix issue with capture callback (now includes post processing results) (#3035)

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

* [HDRP] Fix decal draw order for ShaderGraph decal materials (#3018)

* Fixed ShaderGraph decal draw order

* Updated changelog

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

* Fixed various Look Dev issues after exiting Playmode (#2956)

* Fixed access to invalid Contexts references after exiting playmode.

* Fixed comparison gizmo after playmode.

* Fixes from PR feedback

* StackLit: Fix SG surface option property block to only display energy conserving specular color option for the specular input parametrization (similar to case 1257050) (#3060)

* Fixed missing BeginCameraRendering call for custom render mode of a Camera (#3063)

* Implement custom drawer for layer mask parameters (#3066)

* Adding mixed light baking shadowmask test (#3052)

* adding a shadow mask test

* Update reference images

* Fixing hull constant shader's tesselation cull algorithm.
Before it was comparing edges being out of the frustum. This is wrong
because the camera could be inside a triangle, in which case the edges could
be out of at least 1 frustum plane.
Correct fix is to check all three vertices if they are out of at least 1 plane.
Result packed tightly into the functions w return component.

* Renamed to a new function so we keep backwards compatibility.
Restored the shadow shader pass path, accidentally changed it to bool4 and use near plane.

* Fixing type back to bool, instead of bool4

* Merging changelog information from rebase.

* Making sure scene declaration pass is a bool4

* Adding new test for tesselation culling, with a weird camera angle which fails previous setup.

* Fixing formatting of code, rearranging order of test in editor build settings.

* Adding reference images for test.

* Missing meta file for new tests

* Fix bad merge issue

* fix bad merge issue

* fix bad merge issue

Co-authored-by: JulienIgnace-Unity <julien@unity3d.com>
Co-authored-by: Sebastien Lagarde <sebastien@unity3d.com>
Co-authored-by: FrancescoC-unity <43168857+FrancescoC-unity@users.noreply.github.com>
Co-authored-by: Pavlos Mavridis <pavlos.mavridis@unity3d.com>
Co-authored-by: Antoine Lelievre <antoinel@unity3d.com>
Co-authored-by: slunity <37302815+slunity@users.noreply.github.com>
Co-authored-by: Adrien de Tocqueville <adrien.tocqueville@unity3d.com>
Co-authored-by: Rémi Chapelain <57442369+remi-chapelain@users.noreply.github.com>
…HDRP/fix_1299233_IES_Resize

# Conflicts:
#	com.unity.render-pipelines.core/CHANGELOG.md
#	com.unity.render-pipelines.high-definition/CHANGELOG.md
alelievr and others added 2 commits March 1, 2021 18:15
…HDRP/fix_1299233_IES_Resize

# Conflicts:
#	com.unity.render-pipelines.high-definition/CHANGELOG.md
@sebastienlagarde sebastienlagarde changed the base branch from master to hd/bugfix March 1, 2021 19:38
@sebastienlagarde sebastienlagarde requested review from sebastienlagarde and a team March 1, 2021 19:38
@sebastienlagarde sebastienlagarde changed the base branch from hd/bugfix to master March 1, 2021 19:40
@sebastienlagarde sebastienlagarde marked this pull request as ready for review March 1, 2021 19:40
@sebastienlagarde
Copy link
Contributor

@qa: This PR have already been tested in the past (here we just add gc alloc fix) - #3094 , so merging.

@sebastienlagarde sebastienlagarde merged commit 0db55b2 into master Mar 1, 2021
@sebastienlagarde sebastienlagarde deleted the HDRP/fix_1299233_IES_Resize branch March 1, 2021 19:52
sebastienlagarde added a commit that referenced this pull request Mar 1, 2021
sebastienlagarde added a commit that referenced this pull request Mar 1, 2021
* Hdrp/fix 1299233 ies resize #3243

* Fixed an issue in the planar reflection probe convolution. #3668

* Making the FrameSettingsAttribute field more robust. Using it in other places to avoid caching (#3700)

the Obsoletes. Adding robustness in map query.

* [HDRP] Merge Hd/bugfix #3687

Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: Kleber Garcia <kleber.garcia@unity3d.com>
sebastienlagarde pushed a commit that referenced this pull request Mar 3, 2021
sebastienlagarde added a commit that referenced this pull request Mar 3, 2021
* Fixed a regression introduced when changing the clamping algorithm for RTR and RTGI (case 1307479). (#3206)

* rename occurrences of sample when used as a variable name (#3216)

rename occurrences of sample when used as a shader variable name as this may be a reserved word in certain situatons

* [HDRP] Update eye sample scene #3227

* Change the tooltip for color shadows and semi-transparent shadows (case 1307704). (#3237)

* [Hdrp] Add 3DS max shader material test (#3239)

* Arnold + Physical material

* auto desk interactive

* Reference image from yamato

* linux vulkan test ref image

* Hdrp/fix 1299233 ies resize #3243

* [HDRP] Fix metal warning + fix reference screenshots + disable unstable metal test #3251

* [HDRP] update some docs based on latest changelog (#3253)

* Update some docs

* Proof read upgrading to 2020.2

* Proof read whats new doc

* Fixed typo

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

Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: peterjohnlong <73894199+peterjohnlong@users.noreply.github.com>
Co-authored-by: Rémi Chapelain <57442369+remi-chapelain@users.noreply.github.com>
Co-authored-by: skhiat <55133890+skhiat@users.noreply.github.com>
Co-authored-by: Lewis Jordan <lewis.jordan@hotmail.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.