-
Notifications
You must be signed in to change notification settings - Fork 13
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
SWAPI l2 Implementation - counts to rate #721
SWAPI l2 Implementation - counts to rate #721
Conversation
imap_processing/swapi/l2/swapi_l2.py
Outdated
t_bin time. t_bin is the exposure time per energy bin which is | ||
obtained by dividing the time for one complete sweep | ||
(12 s, coarse + fine sweep) by the total energy steps (72), | ||
i.e., t_bin = 12/72 = 0.167 s. This will be constant. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make this a global constant outside the function? TIME_PER_BIN = 12 / 72
Note that 12/72 is not equal to 0.167, it is 0.166666 repeating. Do you want to store 0.167 or 12/72?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Their algorithm document says to use 0.167
. That's why I used that. This comment was to give background understanding of where that value came from. I can ask SWAPI team this week.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. I left some comments that are more advice about changes that can avoid future challenges.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Some minor nits that don't impact the actual calculations.
00b70ad
into
IMAP-Science-Operations-Center:dev
Change Summary
Overview
Process SWAPI L1 to L2. This PR will close this ticket: #706
New Files
Updated Files
Added L2 updates to these files
Testing