-
Notifications
You must be signed in to change notification settings - Fork 8
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
add nires #781
base: main
Are you sure you want to change the base?
add nires #781
Conversation
Pull Request Test Coverage Report for Build 8075034550Details
💛 - Coveralls |
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.
Looks great, some minor changes requested.
|
||
class UkidssGPS(VizierCatalog): | ||
""" | ||
UKIDSS GPS catalog |
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.
Can you maybe add e.g an arxiv link somewhere to explain what this catalog is?
@@ -0,0 +1,20 @@ | |||
""" | |||
This file contains the configuration for the winter pipeline. |
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.
nires
|
||
def nires_astrometric_ref_catalog_generator(_) -> UkidssGPS: | ||
""" | ||
Generates the astrometric reference catalog for the NIRES image |
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.
returns? Mention UkiddsGps
header[GAIN_KEY] = 1.0 | ||
header["FILTER"] = "ks" | ||
|
||
# header[SNCOSMO_KEY] = sncosmo_filters[header["FILTER"].lower()] |
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.
delete
# header["DATE-OBS"] = header["UTSHUT"] | ||
header["MJD-OBS"] = Time(header["DATE-OBS"]).mjd | ||
|
||
# header.remove("CRVAL1") |
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.
delete (and the others)
if "PROGID" not in header.keys(): | ||
header["PROGID"] = 0 | ||
|
||
if "v240225_0012.fits" in path.as_posix(): |
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.
What is this stuff? Is it needed for the PR?
|
||
def load_raw_nires_image(path: str | Path) -> Image: | ||
""" | ||
Function to load a raw GIT image |
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.
NIRES
No description provided.