pirl-unc / tsarina / 25805197826

Build # 25805197826

Build Type

push

github

Committed by web-flow

Commit Message

Surface filtered as backward-compat alias of passes_filters (#61) (#74)

PR #51 renamed the CTA inclusion column ``filtered`` →
``passes_filters`` (and relaxed the Uncertain/Missing protein-tier RNA
threshold from 0.99 to 0.98). Downstream consumers that still
schema-check for the legacy ``filtered`` column — notably pirlygenes
prior to pirlygenes#241 — silently reject the live tsarina table and
fall back to their own bundled CSV snapshot, which predates PR #51's
threshold change (one-row drift on XAGE1B was the smoking gun in #61).

Add ``filtered`` as a column-level alias for ``passes_filters``
populated at load time in ``_load_cta_dataframe``. Identical dtype and
values; both columns travel together through ``CTA_evidence`` and
``CTA_detailed_evidence``. Pirlygenes' schema check now accepts the
live table and inherits PR #51's semantics without needing the
pirlygenes-side rename (#241) to ship first.

Update the rename-guard tests (``test_csv_has_no_old_columns``,
``test_evidence_uses_passes_filters_column_name``) to reflect the new
contract — ``filtered`` is now a required alias, not a forbidden stale
column. Add a regression test pinning the alias = canonical
equivalence.

Closes #61.

Coverage Stats

2400 of 3390 relevant lines covered (70.8%)

0.71 hits per line

Source File

Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available