-
Notifications
You must be signed in to change notification settings - Fork 3
Karr/finalWorkflow #168
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
base: main
Are you sure you want to change the base?
Karr/finalWorkflow #168
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -270,6 +270,9 @@ class LmRavcSciCentred(BandLmMixin, CgrphAppMixin, SciCentred): | |
| pass | ||
|
|
||
|
|
||
| class LmRavcCentroidTab(BandLmMixin, CgrphAppMixin, CentroidTab): | ||
| pass | ||
|
|
||
| class LmRavcCentroidTab(BandLmMixin, CgrphAppMixin, CentroidTab): | ||
| pass | ||
|
|
||
|
|
@@ -459,3 +462,67 @@ class NCvcSciSnr(BandLmMixin, CgrphAppMixin, SciSnr): | |
|
|
||
| class NCvcPsfMedian(BandLmMixin, CgrphAppMixin, PsfMedian): | ||
| pass | ||
|
|
||
|
|
||
| ### | ||
|
|
||
|
|
||
| class LmAppSciCentred(BandLmMixin, CgrphAppMixin, SciCentred): | ||
| _name_template = r"{band}_APP_SCI_CENTRED" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If this is indeed something that can be parameterized by
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't yet fully understand how the abstraction works, so I might misunderstand. The But then I would expect the same with the band: the |
||
|
|
||
|
|
||
|
|
||
| class LmAppCentroidTab(BandLmMixin, CgrphAppMixin, CentroidTab): | ||
| _name_template = r"{band}_APP_SCI_CENTROID_TAB" | ||
|
|
||
|
|
||
| class LmAppSciSpeckle(BandLmMixin, CgrphAppMixin, SciSpeckle): | ||
| _name_template = r"{band}_APP_SCI_SPECKLE" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciHifilt(BandLmMixin, CgrphAppMixin, SciHifilt): | ||
| _name_template = r"{band}_APP_SCI_HIFILT" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciDerotatedPsfsub(BandLmMixin, CgrphAppMixin, SciDerotatedPsfsub): | ||
| _name_template = r"{band}_APP_SCI_DEROTATED_PSFSUB" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciDerotated(BandLmMixin, CgrphAppMixin, SciDerotated): | ||
| _name_template = r"{band}_APP_SCI_DEROTATED" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciContrastRadprof(BandLmMixin, CgrphAppMixin, SciContrastRadprof): | ||
| _name_template = r"{band}_APP_SCI_RADPROF" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciContrastAdi(BandLmMixin, CgrphAppMixin, SciContrastAdi): | ||
| _name_template = r"{band}_APP_SCI_CONTRAST_ADI" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciThroughput(BandLmMixin, CgrphAppMixin, SciThroughput): | ||
| _name_template = r"{band}_APP_SCI_SCI_THROUGHPUT" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciCoverage(BandLmMixin, CgrphAppMixin, SciCoverage): | ||
| _name_template = r"{band}_APP_SCI_COVERAGE" | ||
|
|
||
|
|
||
|
|
||
| class LmAppSciSnr(BandLmMixin, CgrphAppMixin, SciSnr): | ||
| _name_template = r"{band}_APP_SCI_SNR" | ||
|
|
||
|
|
||
|
|
||
| class LmAppPsfMedian(BandLmMixin, CgrphAppMixin, PsfMedian): | ||
| _name_template = r"{band}_APP_SCI_PSF_MEDIAN" | ||
|
|
||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is exactly the same as the next class; did you mean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hciApp.py was redundant; I temporarily forgot I had defined all the imaging data classes with the first HCI workflow. I have removed it.