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

Added tools and functions to perform energy calibration using sealed sources #5

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

nhgodbole
Copy link

Added Niharika's tools/functions to "calibrate.py".

Wrote tools to calibrate a spectrum from a given file. It's a work in progress, but I needed something to test as a first pass.
Adapted from a Jupyter notebook that does work.
Moved "calibrate_spectrum.py" to "calibration" folder.
Added description and import statements for spectrum_calibration.py.
Added Niharika's spectrum calibration tools/functions and descriptions to "calibration.py".
@ehsteve ehsteve changed the title Added Niharika's tools/functions to "calibrate.py" Added tools and functions to perform energy calibration using sealed sources Sep 24, 2024

"spectrum",
"calibrate_spectrum",
"nearest_energy",
Copy link
Member

Choose a reason for hiding this comment

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

This update breaks the module because you are trying to import a function that you have not defined.

"nearest_energy" is not defined in this code. I goofed -- this was a dummy array that I used in a Jupyter notebook, not a function, so it was never defined in calibration.py.
Added function "gains_offsets" to __all__.
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