Skip to content

DM-51163: Enable multi-tract matching#1286

Open
taranu wants to merge 16 commits into
mainfrom
tickets/DM-51163
Open

DM-51163: Enable multi-tract matching#1286
taranu wants to merge 16 commits into
mainfrom
tickets/DM-51163

Conversation

@taranu
Copy link
Copy Markdown
Contributor

@taranu taranu commented Mar 30, 2026

This also updates various configs to DRP v2 pipelines standards.

doc="The type of sharding (spatial splitting) for the target catalog",
allowed={"tract": "Tract-based shards", "none": "No sharding at all"},
default="tract",
)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is no sharding? Can this work with healpix?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No sharding literally means no sharding at all - no spatial dimension, not even a skymap.

The reason I added this option is some catalogs are small enough that they are entirely within one tract (9813 for COSMOS, 5063 for ECDFS), so it seemed wasteful to have to reingest the same catalog multiple times for skymaps that often have identical tract definitions - especially so if you end up matching two such small external catalogs with each other. Having said that, in retrospect, the benefits aren't that great and I'm not sure if I want to recommend doing it this way.

In principle yes, one could add healpix support, but it would not be completely trivial.

Comment thread python/lsst/pipe/tasks/match_tract_catalog.py Outdated
@taranu taranu force-pushed the tickets/DM-51163 branch from 5481a24 to ba3b14b Compare May 6, 2026 23:02
@taranu taranu force-pushed the tickets/DM-51163 branch from ba3b14b to 8a47b7b Compare June 5, 2026 18:40
@taranu taranu force-pushed the tickets/DM-51163 branch from a7c9949 to 7cde23e Compare June 6, 2026 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants