-
Notifications
You must be signed in to change notification settings - Fork 32
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Editing the change fragment to be properly processed.
- Loading branch information
1 parent
e37085f
commit d34d0fd
Showing
1 changed file
with
6 additions
and
19 deletions.
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,6 @@ | ||
JP-3791: Fixed the `ramp fitting` multiprocessing portion of the | ||
CHARGELOSS recomputation of the readnoise variances. The | ||
DQ array wasn't properly sliced and distributed to the | ||
multiple processes to properly do the recompute. This is | ||
now properly sliced for distribution. | ||
|
||
JP-3771: Added multiple regression tests to the JWST repo. The tests | ||
test multiprocessing in the `jump` step and `ramp_step` and | ||
ensure the outout is the same as during single processing. | ||
This is done by running the same tests as currently being done, | ||
but selecting multiprocessing and comparing output files that | ||
are used for single processing as well. There is an additional | ||
ramp fit multiprocessing test that also saves the optional results | ||
product. | ||
|
||
JP-3708: Added the CRMAG array to the optional results product in `ramp_fit`. | ||
This element of the optional results product had not previously been | ||
implemented in the C-extension. | ||
|
||
This PR fixes JP-3791, JP-3771, and JP-3708. For `ramp_fitting`, the | ||
`CRMAG` element was not originally implementd in the C-extension for | ||
ramp fitting. It is now implemented. The bug the read noise recalculation | ||
for CHARGELOSS when using the multiprocessing option has been fixed. Further, | ||
in `JWST` regression tests have been added to test for multiprocessing to | ||
ensure testing for anything that could affect multiprocessing. |