Tile_qa_plot: add VCCDSEC info and alert #2136
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds in the tile-qa*png plot:
desispec/data/qa/qa-params.yaml
;log.warning()
message is written in that case.We can surely change that threshold=20h value, based on our experience with those events: @schlafly ?
I picked that threshold=20h value based on this plot, which looks at the
b0
campet VCCDSEC values for all exposures since Dec. 2021:(file here: https://data.desi.lbl.gov/desi/users/raichoor/spectro/daily/vccdsec-b0-202112-202310.asc)
Comments:
log.warning()
message is writtendesispec.tile_qa_plot.get_expid_vccdsec() returns a -99 value if the
VCCDSEC` keyword is not present in the frame headerBelow are few examples:
tile-qa-21822-thru20230309.png: that night was discussed here: #2023
tile-qa-25220-thru20231028.png: that recent night also was problematic (desihub/desisurveyops#156)
tile-qa-1863-thru20231031.png: a tile with no problem