DM-37102: convertGaiaXpManager added #300
Conversation
There was a problem hiding this comment.
Note: the PR needs to include the ticket number; see:
https://developer.lsst.io/work/flow.html#make-a-pull-request
Also, the linter failed: that will have to be fixed before merging.
| inFluxKey=item.replace('_','_flux_') | ||
| outFluxKey=item + '_flux' | ||
|
|
||
| inFluxErrKey=item.replace('_','_flux_error_') |
There was a problem hiding this comment.
What is this for? The mag_column_list should have the actual name of the input fields, I would think. What are the names of the input flux fields in the xpSpec files?
There was a problem hiding this comment.
Example xp photometry column names are:
'Lsst_flux_u',
'Lsst_flux_error_u'
I had thought we would want output to be:
'Lsst_u_flux' or 'Lsst_u_fluxErr'
But instead we want to preserve the original column name the fluxes come from in the xp catalog so output will be
'Lsst_flux_u_flux' or 'Lsst_flux_u_fluxErr'
psferguson
changed the title
psferguson
force-pushed
the
tickets/DM-37102
branch
from
824d037
to
6f54eee
Compare
psferguson
force-pushed
the
tickets/DM-37102
branch
from
6f54eee
to
f0c8b88
Compare
psferguson
changed the title
No description provided.