-
Notifications
You must be signed in to change notification settings - Fork 169
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
Split OutlierDetectionStep into multiple steps #8634
Comments
Comment by Ned Molter on JIRA: I started looking into what is needed here on the cal side. It looks like something we'd definitely like to do eventually, and I made a quick draft of what it might look like, which can be found in [this draft pull request|[https://github.com//pull/8722].] However I sort of "jumped the gun" as this needs discussion with INS teams and would require parameter reference file changes, so assigning it to David again for input. |
Now that https://jira.stsci.edu/browse/JP-3347 is resolving without the need to use the IFU algorithm on FS/MOS data (which would have confused this effort significantly) I think we can unblock this. I know at the same time work has been going on to reorganize the documentation, per https://jira.stsci.edu/browse/JP-3342 Tyler Pauly , pushing this back over to you. Is this still desirable from your side based on the recent doc updates? |
Comment by Ned Molter on JIRA: IMO this is still worth doing, although the documentation updates make it less urgent/crucial. I'm going to resurrect my (now ancient) PR for this and see if the changes do indeed look helpful to folks. |
Issue JP-3682 was created on JIRA by Brett Graham:
The outlier detection step currently handles several different "modes" in sometimes very different ways:
The docs mention that the parameters for the step are different depending on the mode:
https://jwst-pipeline.readthedocs.io/en/latest/jwst/outlier_detection/arguments.html
The step spec includes many items that are entirely unused in some modes (which is not fully documented).
These modes also expect very different input types and produce different results.
Different modes use different file suffixes, none of which are currently handled by the shared suffix machinery.
It seems reasonable that these should be split into different steps. However this will require:
pars-
references for the new stepspars-outlierdetectionstep
The text was updated successfully, but these errors were encountered: