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
DM-30858: Add GAaP to obs_subaru #367
Conversation
config/forcedPhotCcd.py
Outdated
@@ -2,6 +2,7 @@ | |||
|
|||
config.measurement.load(os.path.join(os.path.dirname(__file__), "apertures.py")) | |||
config.measurement.load(os.path.join(os.path.dirname(__file__), "kron.py")) | |||
config.measurement.load(os.path.join(os.path.dirname(__file__), "gaap.py")) |
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.
Don't turn on here:
- no slow measurement plugin on any per-visit processing.
- Even if it were fast, I'm not sure it makes sense.
Let's talk about it first.
config/gaap.py
Outdated
@@ -0,0 +1,7 @@ | |||
# Enablue GAaP (Gaussian Aperture and PSF) colors | |||
# 'config' is a SourceMeasurementConfig |
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.
This comment isn't universally true
config/gaap.py
Outdated
@@ -0,0 +1,7 @@ | |||
# Enablue GAaP (Gaussian Aperture and PSF) colors |
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.
Enablue to Enable
config/measureCoaddSources.py
Outdated
@@ -4,6 +4,7 @@ | |||
config.measurement.load(os.path.join(os.path.dirname(__file__), "kron.py")) | |||
config.measurement.load(os.path.join(os.path.dirname(__file__), "convolvedFluxes.py")) | |||
config.measurement.load(os.path.join(os.path.dirname(__file__), "hsm.py")) | |||
config.measurement.load(os.path.join(os.path.dirname(__file__), "gaap.py")) |
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.
Don't turn on here.
policy/Object.yaml
Outdated
# PSF GAaP flux | ||
GaapPsFlux: | ||
functor: NanoJansky | ||
dataset: meas |
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.
dataset: forced_src
if we're running only on forcedPhotCoadd.
I got a comment on my email about not wanting to run gaap in |
71f086a
to
e2ebf0d
Compare
eebce0f
to
64ef76e
Compare
64ef76e
to
0d646ce
Compare
No description provided.