You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I can see the benefit of having these be just ascii (they are human readable when the file is opened with a text editor). If this restriction is ok than the standard should be updated to reflect that these bytes must be ascii.
The text was updated successfully, but these errors were encountered:
Here is one example:
asdf/asdf/compression.py
Line 183 in ecb4d4f
The standard doesn't mention that these 4 bytes should be limited to the ascii range:
https://asdf-standard.readthedocs.io/en/1.0.3/file_layout.html#block-header
Modifying the LzmaCompressor used in
test_compression_with_extension
to use label `b"lzm\255" results in a failure at:asdf/asdf/compression.py
Line 201 in ecb4d4f
I can see the benefit of having these be just ascii (they are human readable when the file is opened with a text editor). If this restriction is ok than the standard should be updated to reflect that these bytes must be ascii.
The text was updated successfully, but these errors were encountered: