Skip to content

Keep a fixed obs column order in from_scanpy - #1162

Merged
Zethson merged 1 commit into
mainfrom
fix/coda-from-scanpy-obs-order
Oct 2, 2026
Merged

Zethson merged 1 commit into
mainfrom
fix/coda-from-scanpy-obs-order

Conversation

@Zethson

@Zethson Zethson commented Oct 2, 2026

Copy link
Copy Markdown
Member

from_scanpy deduplicates sample identifiers and covariates order-preservingly instead of via a set, so coda.obs columns come out in a fixed order independent of PYTHONHASHSEED.
Values are unchanged.

Sample identifiers and covariates were deduplicated through a set, so the order of coda.obs columns depended on PYTHONHASHSEED.
Use an order-preserving dict.fromkeys with sample identifiers first; values are unchanged.
@codecov-commenter

codecov-commenter commented Oct 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.69%. Comparing base (72498c0) to head (180bfd2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1162      +/-   ##
==========================================
+ Coverage   75.67%   75.69%   +0.02%     
==========================================
  Files          55       55              
  Lines        8431     8431              
==========================================
+ Hits         6380     6382       +2     
+ Misses       2051     2049       -2     
Files with missing lines Coverage Δ
src/pertpy/tools/_coda/_base_coda.py 61.09% <100.00%> (+0.31%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Zethson
Zethson merged commit c0319ca into main Oct 2, 2026
23 checks passed
@Zethson
Zethson deleted the fix/coda-from-scanpy-obs-order branch October 2, 2026 10:59
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