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-31825: Add columnar access to source tables and support new table column names #193
Conversation
python/lsst/jointcal/jointcal.py
Outdated
visitCatalog = inputSourceTableVisit[catalogMap[visitSummaryRef.dataId['visit']]].get() | ||
|
||
dataRef = inputSourceTableVisit[catalogMap[visitSummaryRef.dataId['visit']]] | ||
if isinstance(dataRef, dafPersist.ButlerDataRef): |
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 shouldn't be necessary here at all: this method is only called when running in gen3.
217f2b3
to
fdea28c
Compare
@@ -885,15 +884,24 @@ def _load_data(self, inputSourceTableVisit, inputVisitSummary, associations, | |||
catalogMap = {ref.dataId['visit']: i for i, ref in enumerate(inputSourceTableVisit)} | |||
detectorDict = {detector.getId(): detector for detector in camera} | |||
|
|||
columns = None |
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.
Please add a note here about how this is filled out with the columns to be load from the sourceTable during the first loop iteration.
@@ -920,6 +928,53 @@ def _make_one_input_data(self, visitRecord, catalog, detectorDict): | |||
filter=lsst.afw.image.FilterLabel(band=visitRecord['band'], | |||
physical=visitRecord['physical_filter'])) | |||
|
|||
def _get_sourceTable_visit_columns(self, inColumns): | |||
""" | |||
Get the sourceTable_visit columns from the config. |
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.
I wonder if this is something we can put somewhere more general? I don't know if a tweak of this is generally applicable for code that reads the visit-level tables?
fdea28c
to
e4b2fcf
Compare
No description provided.