RampFitStep - accept integer values for maximum_cores

[izkgao]

This PR allows maximum_cores of RampFitStep to accept integer values for the number of cores to be used in multiprocessing. Documentation updates follow the changes in #7871.

Checklist for maintainers

• added entry in CHANGES.rst within the relevant release section
• updated or added relevant tests
• updated relevant documentation
• added relevant milestone
• added relevant label(s)
• ran regression tests, post a link to the Jenkins job below.
  How to run regression tests on a PR
• Make sure the JIRA ticket is resolved properly

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1e8b366) 75.24% compared to head (9744dd1) 75.24%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #8123   +/-   ##
=======================================
  Coverage   75.24%   75.24%           
=======================================
  Files         470      470           
  Lines       38420    38420           
=======================================
  Hits        28908    28908           
  Misses       9512     9512           

Flag	Coverage Δ		*Carryforward flag
nightly	77.37% <ø> (ø)		Carriedforward from 1e8b366

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hbushouse]

@mwregan2 What are your thoughts on this suggested update? If it's OK for ramp_fit, we should probably make the same change to jump too.

[izkgao]

@hbushouse The same change has been done to jump by #7871.

[mwregan2]

@mwregan2 What are your thoughts on this suggested update? If it's OK for ramp_fit, we should probably make the same change to jump too.

I am confused with this. We seem to have lost the change that I put in for ramp fit at the same time I did it for jump.

[kmacdonald-stsci]

@mwregan2 What are your thoughts on this suggested update? If it's OK for ramp_fit, we should probably make the same change to jump too.

I am confused with this. We seem to have lost the change that I put in for ramp fit at the same time I did it for jump.

I believe your previous changes were only in STCAL: spacetelescope/stcal#183

[mwregan2]

Well, I did make JWST changes but they seem to have been lost. I probably was at fault.

[hbushouse]

Given that this change was intended to be implemented before (but apparently got lost) and appears to be exactly the same updates made to the jump step to enable the use integer values for the number of cores, I think this all seems very reasonable and we should adopt it. I've started a regression test run against this PR branch at https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1113 to make sure it's all working OK.

[hbushouse]

Regression test differences are all unrelated (related to MIRI emicorr).