Skip to content

Commit

Permalink
Fixup some tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jeromekelleher committed May 2, 2024
1 parent 347d92c commit 0285994
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
3 changes: 1 addition & 2 deletions bio2zarr/vcf.py
Original file line number Diff line number Diff line change
Expand Up @@ -2056,8 +2056,7 @@ def get_max_encoding_memory(self):
"call_genotype_mask",
]
gt_mem = sum(
self.schema.fields[col].variant_chunk_nbytes
for col in encoded_together
self.schema.fields[col].variant_chunk_nbytes for col in encoded_together
)
return max(max_encoding_mem, gt_mem)

Expand Down
4 changes: 2 additions & 2 deletions tests/test_core.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,8 +188,8 @@ def test_5_chunk_1(self, n, expected):
# It *might* work in CI, but it may well not either, as it's
# probably dependent on a whole bunch of things. Expect to fail
# at some point.
("tests/data", 4630726),
("tests/data/vcf", 4618589),
("tests/data", 4960266),
("tests/data/vcf", 4948129),
("tests/data/vcf/sample.vcf.gz", 1089),
],
)
Expand Down
2 changes: 1 addition & 1 deletion tests/test_vcf_examples.py
Original file line number Diff line number Diff line change
Expand Up @@ -881,7 +881,7 @@ def test_split_explode(tmp_path):
vcf.explode_partition(out, j)
vcf.explode_finalise(out)
pcvcf = vcf.IntermediateColumnarFormat(out)
assert pcvcf.columns["POS"].vcf_field.summary.asdict() == {
assert pcvcf.fields["POS"].vcf_field.summary.asdict() == {
"num_chunks": 3,
"compressed_size": 587,
"uncompressed_size": 1008,
Expand Down

0 comments on commit 0285994

Please sign in to comment.