[Bug]: Gwy file not loading

[SylviaWhittle]

Checklist

• [X] Re-run analysis with topostats process --core 1.
• [X] Describe the bug.
• [X] Include the configuration file.
• [X] Copy of the output.
• [X] The exact command that failed. This is what you typed at the command line, including any options.
• [X] TopoStats version, this is reported by topostats --version
• [X] Operating System and Python Version

Describe the bug

Some .gwy files fail to load.

Copy of the output

Include the configuration file

Default config with .gwy selected.

To Reproduce

Run TopoStats on this file: https://drive.google.com/file/d/1CWQnVYJg0UXuJLqZ4xWYbnjDM5v2u5N-/view?usp=share_link. Not to be shared outside the University.

TopoStats Version

Git main branch

Python Version

3.11

Operating System

MacOS M1/M2 (post-2021)

Python Packages

N/A

[SylviaWhittle]

I believe I have found the root cause and a solution.

The crash is caused by the byte 0xb5 (the micro symbol) not being able to be read by read_null_terminated_string. This is due to said byte not being in the utf-8 encoding that is default for the function. I resolved this by using the latin1 encoding which is able to handle said character.

I think the fix should try utf-8 as it supports more characters than latin1, but to try latin1 after a failed read of utf-8. Branch & commit to follow.