Skip to content

Commit

Permalink
fix: spec mask, AND_MASK -> OR_MASK
Browse files Browse the repository at this point in the history
fix as per @Sean-Morrison 's suggestion in astropy pull req [astropy#1107](astropy#1107)

could be reverted in future, in which case this commit can just be deleted
  • Loading branch information
rileythai committed Nov 18, 2023
1 parent fd999ce commit 1a7b519
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions specutils/io/default_loaders/sdss_v.py
Original file line number Diff line number Diff line change
Expand Up @@ -401,9 +401,7 @@ def _load_BOSS_HDU(hdulist: HDUList, hdu: int, **kwargs):
try:
mask = hdulist[hdu].data["MASK"]
except KeyError:
# TODO: find out which or AND_MASK or OR_MASK is wanted
# i'm assuming AND_MASK is more stringent, so AND it is
mask = hdulist[hdu].data["AND_MASK"]
mask = hdulist[hdu].data["OR_MASK"]

# Fetch metadata
# NOTE: specFull file does not include S/N value, but this gets calculated
Expand Down

0 comments on commit 1a7b519

Please sign in to comment.