Display all detectors #1213
Display all detectors #1213
Conversation
|
Note: PR #1197 should be merged before this one. |
melanieclarke
requested review from
mfixstsci,
BradleySappington and
bhilbert4
and removed request for
BradleySappington
There was a problem hiding this comment.
This looks great, and was a ton of work! I know the NIRCam team will be excited to be able to see all the detectors on a single page.
@mfixstsci I gave everything a look except for the js updates.
There was a problem hiding this comment.
Most of these changes are stylistic, this is a ton of great work!
One overall issue is that the default thumbnail (which I believe should be changed to imagenotfound.png) is not being used for your groups or files if the thumbnail does not exist.
Example here if loaded on test:
https://jwql-test.stsci.edu/NIRCam/archive/2057/obs003/
|
Thank you for submitting this @melanieclarke and thank you @BradleySappington for the review. Merging! |
On the observation pages, there is now an option to group by exposure or file. For instruments with multiple detectors, grouping by exposure pulls up a page where previews from all simultaneous detector data are displayed together. From this page, you can mark an anomaly to apply to the whole group of files, or mark the whole group as Viewed. The preview images in the exposure group page are clickable: they will take you to the individual image page for that file, which works the way it always has.
For instruments with no simultaneous detectors, the exposure grouping is allowed, but has no real effect. The exposure pages have equivalent functionality to the existing image pages.
The archive date image page and JWQL query page also allow grouping by exposure.