We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Per the spec, Char fields should have dtype |S1, but we are currently outputting "<U1", e.g.
|S1
cat field_type_combos.vcf.vcz/variant_IC1/.zarray { "chunks": [ 10000 ], "compressor": { "blocksize": 0, "clevel": 7, "cname": "zstd", "id": "blosc", "shuffle": 0 }, "dimension_separator": "/", "dtype": "<U1", "fill_value": null, "filters": null, "order": "C", "shape": [ 208 ], "zarr_format": 2 }
The text was updated successfully, but these errors were encountered:
Fix Char type bug
136e41f
Change U1 to S1 as per spec Closes sgkit-dev#268
Ah yes - it's not actually clear we want to do this: sgkit-dev/vcf-zarr-spec#14
I think it would be simpler if we standardised on UTF8 going forward, so I'm going to close this as a "wonfix"
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
Per the spec, Char fields should have dtype
|S1
, but we are currently outputting "<U1", e.g.The text was updated successfully, but these errors were encountered: