Tom White
Tom White
> I think it might make sense for me to remove the `CubedManager` class from this PR and instead put that & cubed+xarray tests into another repo. That keeps xarray's...
> I would like to get to the point where you can use xarray with a chunked array **without ever importing dask**. I think this PR gets very close, but...
Thanks for all your hard work on this @TomNicholas!
This originally came up here: https://github.com/pystatgen/sgkit/issues/463#issuecomment-827445369
@jeromekelleher we used to use the "#CHROM POS.." line to support round-tripping of VCF -> Zarr -> VCF, but we can generate the header now, so it may not be...
> I noticed that for some reason https://sgkit-dev.github.io/sgkit is redirecting to https://pystatgen.github.io/sgkit/latest, and I'm not quite sure where that rule is encoded. It's in the gh-pages branch: https://github.com/sgkit-dev/sgkit/blob/gh-pages/index.html > It...
Sounds good - `all_fields.vcf` is not a VCF from the wild, so it's OK to change it.
See https://github.com/pystatgen/sgkit/blob/main/sgkit/tests/io/vcf/test_vcf_generator.py
Is changing the default in https://github.com/pystatgen/sgkit/blob/main/sgkit/accelerate.py#L6 sufficient?
Closing this as [bio2zarr](https://github.com/sgkit-dev/bio2zarr) is now established.