Howard Bushouse

Results 116 comments of Howard Bushouse

> https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1283/ But it is full of errors. There are so many it is hard to pull out which ones are related to this PR I see a lot of...

I think we need to wait to merge this until another jwst PR gets merged, which does an update to either the jump or ramp_fit steps, so that it's compatible...

@mwregan2 Now that https://github.com/spacetelescope/stcal/pull/242 has been merged, this PR is the one that's needed to enable the use of the new parameter, right?

> Right OK, I'll kick off a regression test run using this PR branch and the latest stcal/main, which contains the necessary updates on that side.

Regression test started at https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1327

First attempt at regtest was messed up. New run started at https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1331/

Finally got a successful regtest run at https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1341/ All differences are expected due to the updates here, as well as unrelated ref file updates. So it looks OK.

This should not be merged until there's consensus as to whether the warning message is accurate (i.e. I don't think it's benign, as has been suggested).

> One more thing would make this PR perfect: set default weight to exptime, especially given that #7563 still hasn't been approved. That's going to be done via the use...

@nden @jdavies-st Any final comments or suggestions before we merge this?