Add algorithm parameter to ramp_fit spec about jwst HOT 7 CLOSED

Comment by Howard Bushouse on JIRA:

Fixed by #8503

from jwst.

Comment by David Law on JIRA:

I saw that stcal had been tagged recently, but is jwst pointing to stcal 1.7.1 yet?

from jwst.

Comment by Howard Bushouse on JIRA:

David Law the latest jwst requirements file has stcal pinned to "stcal>=1.7.0,<1.8.0", so while it doesn't yet require stcal 1.7.1 as a minimum, it can be installed with the current jwst/master.

from jwst.

Comment by David Law on JIRA:

Howard Bushouse At the moment it's quite confusing though; 1.7.0 is used by default, which will happily accept the algorithm='OLS_C' parameter, print out 'Using algorithm = OLS_C', but then not actually use the C algorithm and keep using the python code instead.  If we could update to require 1.7.1 minimum it will be easier to point people at this code to test (in addition to bringing the poisson error fixes into latest dev).

from jwst.

Comment by Howard Bushouse on JIRA:

David Law Agreed. Will do.

from jwst.

Comment by Howard Bushouse on JIRA:

David Law  stcal min bumped to 1.7.1 in #8505 So it should be good to go now when using jwst/master.

from jwst.

Comment by David Law on JIRA:

Looks good to me, tested and working as expected.

from jwst.