Skip to content
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

NaN Pixels, buffer corruption #1664

Open
ConsueTerra opened this issue Nov 3, 2023 · 1 comment
Open

NaN Pixels, buffer corruption #1664

ConsueTerra opened this issue Nov 3, 2023 · 1 comment
Labels
backport-me A PR that is to be backported to the latest stable Chunky version. bug renderer

Comments

@ConsueTerra
Copy link
Contributor

Under certain scenes, pixels that appear corrupted arise. There are pure white pixels in the viewport as well as pure black pixels. At some point those pixels got corrupted while rendering, the can either be NaN pixels, or +- infinity pixels Im not sure (since white and black pixels seem to be co present). Now this introduces small visual artifacts, but the bigger issue is that when exporting the raw floating point data like in TIFF32 or PFM modes, the NaN pixels are also written. As such programs downstream that cannot deal with NaN values properly will fail to load the image (like in Adobe Camera RAW) or would fail to save the image properly (as in Photoshop). This could be considered a critical bug as there is no way to get around this issue atm and unclear how to stop a scene buffer from corrupting.

Bug observed:
2.4.5 (reproduced here)
2.5.0-SNAPSHOT.214.g94622f8a (commit 94622f8)
#1660

Example

default_2022-12-04_17-28-33-2 4 5-2956
Here you see the white pixels in the context of the render (exposure has been tuned down)
default_2022-12-04_17-28-33-2 4 5-2960
Here I turned off post and set exposure to -300, the white pixels remain
default_2022-12-04_17-28-33-2 4 5-3064
I set exposure to 1000, to my surprise there where also black pixels.

Here is a screenshot from the photos app, showcasing the apparent corruption
image

In Lightroom, showing 2(3) renders under different export options and different samples. Some of the images are corrupted and unloadable
image

Reproduction turned out easier than I expected, I have included the 2.4.5 version. The original scene was made in a snapshot build but I have lost the JSON for it so I will have to see in I can remake that scene. Note for this scene under #1660 no corruption was observed but WAS for a different scene (That motivated this issue) but it took considerably longer for problems to occur so decided to use this scene as a benchmark.
Link to 2.4.5 scene with NaN pixels

@leMaik leMaik added bug renderer backport-me A PR that is to be backported to the latest stable Chunky version. labels Nov 3, 2023
@ThatRedox
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-me A PR that is to be backported to the latest stable Chunky version. bug renderer
Projects
None yet
Development

No branches or pull requests

3 participants