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.

Sign up for GitHub

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

Jump to bottom

Fix warning at startup due to uninitilized memory #114

Merged

jaagut merged 1 commit into main from Flova-patch-1

Oct 28, 2023

Member

Flova commented Oct 27, 2023 •

edited

Loading

Proposed changes

Use np.zeros instead of empty. This is slightly slower, but we don't have overflows due to weird initializations which are transformed.
Closes Invalid values and overflow #72


          Fix warning at startup due to uninitilized memory

3bd4c6f

Flova requested review from jaagut, phinik and val-ba

October 27, 2023 14:37

jaagut approved these changes

View reviewed changes

jaagut merged commit 6003b88 into main

4 checks passed

jaagut deleted the Flova-patch-1 branch

October 28, 2023 12:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet