Skip to content

Reinstate VariantData error checks#1106

Open
hyanwong wants to merge 1 commit intotskit-dev:mainfrom
hyanwong:reinstate-vdata-checks
Open

Reinstate VariantData error checks#1106
hyanwong wants to merge 1 commit intotskit-dev:mainfrom
hyanwong:reinstate-vdata-checks

Conversation

@hyanwong
Copy link
Member

@hyanwong hyanwong commented Mar 3, 2026

Using tskit2zarr rather than sgkit

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.51%. Comparing base (a72866c) to head (5599d26).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1106      +/-   ##
==========================================
+ Coverage   90.22%   90.51%   +0.28%     
==========================================
  Files          19       19              
  Lines        7092     7092              
  Branches     1170     1170              
==========================================
+ Hits         6399     6419      +20     
+ Misses        562      551      -11     
+ Partials      131      122       -9     
Flag Coverage Δ
C 81.41% <ø> (ø)
c-python 59.43% <ø> (ø)
python-tests 96.13% <ø> (+0.46%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Python API 96.25% <ø> (+0.46%) ⬆️
Python C interface 66.66% <ø> (ø)
C library 88.58% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hyanwong hyanwong force-pushed the reinstate-vdata-checks branch from 81325e2 to 661bd48 Compare March 3, 2026 13:08
@jeromekelleher
Copy link
Member

Best to hold off on this until Zarr v3 has landed in #1104

Using tskit2zarr rather than sgkit
@hyanwong hyanwong force-pushed the reinstate-vdata-checks branch from 661bd48 to 5599d26 Compare March 3, 2026 13:40
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