UnicodeDecode Error about galvani

[chatcannon]

I wasn't sure what encoding to use because I have never had an MPT file with non-ASCII characters, and I no longer have access to a Bio-Logic instrument to create test files. I thought it would be better to raise an error than to produce nonsense data by choosing the wrong encoding.

Try replacing the encoding='ascii' with encoding='utf8' and see if that works and the text comes out correctly. If not then try whatever character encoding is most often used in your locale e.g. 'latin1' for Western Europe.

Alternatively send me the file or attach it to this issue and I will have a go.

[chatcannon]

P.S. If encoding='utf8' works then tell me so that I can change to that in the next release.

[linuxrider]

Had same problem with the attached file.
Had to rename extension due to github limitations.
galvani test.txt

latin1 is a solution in this case.
Does it make sense to optionally state the encoding as kwarg?
The situation with this file is more complicated though, will open another issue for that.

[chatcannon]

Does it make sense to optionally state the encoding as kwarg?

Yes it does, this will be easier if I no longer have to support both python 2 and python 3 so maybe it is worth bringing forward #15 together with this

[marionbregeault]

Hi, Yes 'latin1' worked for me ! But not utf8. Thank you!