Skip to content

Bugfix/nan#53

Merged
Verasl merged 2 commits intorelease/2019.3from
bugfix/nan
Sep 3, 2019
Merged

Bugfix/nan#53
Verasl merged 2 commits intorelease/2019.3from
bugfix/nan

Conversation

@hkr
Copy link
Contributor

@hkr hkr commented Aug 29, 2019

Changes to avoid NaN with HDR rendering on Android devices.

  • sanitized the BushTest normal map
  • add normalize node to Vegetation shader (this could probably also be fixed by changing the normal map, but I'm not sure)

Note: more fixes coming to SRP repo

hkr added 2 commits August 29, 2019 10:11
Filled with (0, 0, 1) where alpha was < 1
Re-normalized all normals and replaced invalid values with (0, 0, 1)
Set alpha channel to 1
This normalize should not be necessary, but the specific normal map of lower LODs of the palm trees otherwise generate NaNs during tangent to world space transformation.
@hkr hkr requested a review from Verasl August 29, 2019 17:22
Copy link
Contributor

@Verasl Verasl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Verasl Verasl merged commit 5ee2c75 into release/2019.3 Sep 3, 2019
@Verasl Verasl deleted the bugfix/nan branch September 3, 2019 12:13
Verasl added a commit that referenced this pull request Feb 4, 2020
* Updated burst package, fixed missing renderer shaders

* Update package to public 7.0.0 package

* Update BushTest normal map

Filled with (0, 0, 1) where alpha was < 1
Re-normalized all normals and replaced invalid values with (0, 0, 1)
Set alpha channel to 1

* Vegetation ShaderGraph: Normalize normal before master node

This normalize should not be necessary, but the specific normal map of lower LODs of the palm trees otherwise generate NaNs during tangent to world space transformation.

* Bugfix/nan (#53)

* Update BushTest normal map

Filled with (0, 0, 1) where alpha was < 1
Re-normalized all normals and replaced invalid values with (0, 0, 1)
Set alpha channel to 1

* Vegetation ShaderGraph: Normalize normal before master node

This normalize should not be necessary, but the specific normal map of lower LODs of the palm trees otherwise generate NaNs during tangent to world space transformation.

* Update README.md

* Update LICENSE

changed Boat Attack License to UCL

* Rename LICENSE to LICENSE.md

* Fixed typo in welcome screen

Co-authored-by: Florian Penzkofer <fp@nullptr.de>
Verasl added a commit that referenced this pull request Feb 10, 2020
* Fixed typo in welcome screen

* Release/2019.3 (#68)

* Updated burst package, fixed missing renderer shaders

* Update package to public 7.0.0 package

* Update BushTest normal map

Filled with (0, 0, 1) where alpha was < 1
Re-normalized all normals and replaced invalid values with (0, 0, 1)
Set alpha channel to 1

* Vegetation ShaderGraph: Normalize normal before master node

This normalize should not be necessary, but the specific normal map of lower LODs of the palm trees otherwise generate NaNs during tangent to world space transformation.

* Bugfix/nan (#53)

* Update BushTest normal map

Filled with (0, 0, 1) where alpha was < 1
Re-normalized all normals and replaced invalid values with (0, 0, 1)
Set alpha channel to 1

* Vegetation ShaderGraph: Normalize normal before master node

This normalize should not be necessary, but the specific normal map of lower LODs of the palm trees otherwise generate NaNs during tangent to world space transformation.

* Update README.md

* Update LICENSE

changed Boat Attack License to UCL

* Rename LICENSE to LICENSE.md

* Fixed typo in welcome screen

Co-authored-by: Florian Penzkofer <fp@nullptr.de>

* fixes

- fixed precision error in checkpoint shader
- fixed tangents on clouds
- fixed touch controls appearing on switch
- clamped skybox sun
- changed reflection probes to baked

* Fixed Issue with addressables

* Added back Heightmap generation for the Water

* fixed DX11 sampler

Co-authored-by: Florian Penzkofer <fp@nullptr.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants