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

sky_sampling test converges incorrectly with large amount of samples. #2

Open
nonl4331 opened this issue Oct 25, 2022 · 0 comments
Open
Labels
bug Something isn't working

Comments

@nonl4331
Copy link
Owner

The sky sampling test checks if the numerically integrated solution from the generated PDF matches the sampled solution. With large amounts of samples (> 10^6) it becomes evident that while they are very close there are slight differences that cannot be attributed to noise.

The test fails in this case as expected. While the test does fail the image produced should still be unbiased since the sampled pdf does appear to integrate to 1. Furthermore, since the sampled result is similar to the numerically integrated solution this should have a negligible impact on performance.

It is likely that this issue is due to floating-point errors. It is unclear from which bit of the code the error is coming from.

@nonl4331 nonl4331 added the bug Something isn't working label Oct 25, 2022
@nonl4331 nonl4331 changed the title sky_sampling test converges incorrectly with large ammount of samples. sky_sampling test converges incorrectly with large amount of samples. Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant