Skip to content

convert energies to float64 before summing them in analysis script#142

Merged
KrisThielemans merged 1 commit intoETSInitiative:mainfrom
gschramm:gschramm-patch-1
Jul 16, 2025
Merged

convert energies to float64 before summing them in analysis script#142
KrisThielemans merged 1 commit intoETSInitiative:mainfrom
gschramm:gschramm-patch-1

Conversation

@gschramm
Copy link
Collaborator

Changes in this pull request

  • fix floating point precision issues when summing energies of many events

Testing performed

Related issues

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added docstrings/doxygen in line with the guidance in the developer guide
  • The code builds and runs on my machine

Contribution Notes

Please read and adhere to the contribution guidelines.

Please tick the following:

  • The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in the ETSI software (the Work) under the terms and conditions of the Apache-2.0 License.

@KrisThielemans KrisThielemans merged commit 08991d4 into ETSInitiative:main Jul 16, 2025
2 checks passed
@KrisThielemans KrisThielemans self-assigned this Jul 16, 2025
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.

float precision error in avg. energy calc. of analysis script

2 participants