Hello@effigies, Thank you for updating!

• In the filenibabel/surfice.py:

Line 175:1:W293 blank line contains whitespace

To test for issues locally,pip install flake8 and then runflake8 nibabel.

Comment last updated at 2021-11-19 13:45:48 UTC

Codecov Report

❌ Patch coverage is29.53020% with105 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.91%. Comparing base (db4c859) to head (cb9a77a).
⚠️ Report is 58 commits behind head on master.

@@            Coverage Diff             @@##           master    #1066      +/-   ##==========================================- Coverage   95.34%   94.91%   -0.43%==========================================  Files         209      210       +1       Lines       29777    29917     +140       Branches     3357     3380      +23     ==========================================+ Hits        28390    28396       +6- Misses        948     1073     +125- Partials      439      448       +9

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report?Share it here.

@effigies thanks for supporting this. Themz3 specification is here.

• It is true that a single extension is used for both compressed and uncompressed data. You can discriminate these by looking at the first two bytes of the file. Specifically, all GZip files begin with hex = 0x1F8B, all uncompressed MZ3 files begin with hex = 0x4D5A
• You do not need to know the file size to compute the array size. The headerNFACE andNVERT explicitly describe the number of indexed triangles (faces) and spatial points (vertices).

Happy to work directly with you to tidy this up.