forked from spacetelescope/stcal
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Get up to date May 24, 2022 #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
To avoid the dependency on the JWST package. I deleted the unit test.
Fix for jumps in first good group after dropping groups
JP-2556: Handle charge spilling out of saturated pixels into neighbors
prep for 0.6.4
Removing whitespaces. Adding zframe as a return value from flag_saturated_pixels. Moving ZEROFRAME handling to a separate function. Finished adding ZEROFRAME to the linearity module, including unit testing, as well as corrected style issues. Updating saturation. Adding processing to use ZEROFRAME data instead of SCI data if the ramp is saturated, but the ZEROFRAME data is good. Removed print statements used for test development. Adding PR line to the change log for usage of ZEROFRAME. Updating based on code review. Changing the detection of ZEROFRAME usage. Adding comments for testing. Removing unnecessary comment. Fixing code style failures. Created a function to compute the median rates that also accounts for using the ZEROFRAME. Added frame time computation for ZEROFRAME data. Adding ZEROFRAME testing to ramp fitting unit tests. Making changes due to code review. Computational errors were found during code review, so tests needed to be modified for computation corrections.
…e the adjacency saturation with ZEROFRAME.
JP-374: Use ZEROFRAME for Saturated Ramps.
JP-2581: fix to allow multiprocessing for jump
…amps. (#92) * Updating the handler for the suppression of one good group ramps. * Changing the change log for the ramp fitting update. * Changing function name.
update change log for 0.7.3
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.