Closed johnkerl closed 4 weeks ago
Leaving in draft while I address a unit-test failure
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 89.79%. Comparing base (
4a13a82
) to head (265c9b4
). Report is 1 commits behind head on main.
Issue and/or context: Pre-existing bug found while working on issue #2407 / [sc-51048].
I ran
sdf.count()
on a variant-indexed dataframe which has dim0 being a string (which the spec allows even though most users don't take advantage of this). I got the errorThere is a repro now built into the unit-test cases on this PR.
Changes:
Only use the code that assumes dim0 is
int64
soma_joinid
when that is in fact the case.Notes for Reviewer: