Skip to content

feat(query): filter on the tissue and disease roll-up columns - #73

Merged
timtreis merged 1 commit into
mainfrom
feat/facet-anchor-columns
Nov 9, 2026
Merged

timtreis merged 1 commit into
mainfrom
feat/facet-anchor-columns

Conversation

@timtreis

@timtreis timtreis commented Nov 9, 2026

Copy link
Copy Markdown
Member

The MCP connector (spatialdata-db-web #99) gives snippets such as Catalog(version="2026-11-08").query(tissue_organ="brain", validation=None). query() rejected them, because tissue_system, tissue_organ, disease_class and disease_group were catalog columns but not facets.

Changes:

  • Added the four columns to FACETS. The hub cut writes each row's anchor, so equality on an anchor includes every tissue or disease under it.
  • Added a test with the connector's form, case aliasing, a combined filter and a misspelt value.

Checks:

  • 471 tests pass.
  • On release 2026-11-08 (12,159 datasets), the connector's forms run with validation=None. For example, tissue_organ="brain" gives 4,022 datasets and organism="human", tissue_organ="brain", technology="Xenium" gives 164.

Users who install from the v0.1.3 tag get this only after a new tag.

query() rejected tissue_system, tissue_organ, disease_class and disease_group, although the hub
cut publishes them and the MCP connector's snippets filter on them, e.g.
Catalog(version="2026-11-08").query(tissue_organ="brain", validation=None). They are now facets.
Each row carries its anchor, so equality on an anchor includes every tissue or disease under it.
@codecov

codecov Bot commented Nov 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.84%. Comparing base (e852300) to head (2f7a924).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #73   +/-   ##
=======================================
  Coverage   94.84%   94.84%           
=======================================
  Files          22       22           
  Lines        2326     2326           
=======================================
  Hits         2206     2206           
  Misses        120      120           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 spatialdata-db | 🛠️ Build #35763351 | 📁 Comparing 2f7a924 against latest (e852300)

  🔍 Preview build  

1 file changed
± reference/catalog-columns.html

@timtreis
timtreis merged commit 11b6780 into main Nov 9, 2026
9 of 10 checks passed
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.

1 participant