Simon Kern

Results 155 comments of Simon Kern

Awesome! Is there a possibility to get the other feature on the roadmap? 😇 It was really useful in Everything that after selecting a certain file from a query, next...

I changed using `@master` to `@v3` and `@v4` in GitHub actions as suggested generally in https://github.com/actions/upload-artifact/issues/41 (not specifically for Python setup in GitHub actions, but is probably generally better to...

Went once through all the changes, added some more comments. I think all changes are okay and it's long overdue to merge this PR Thanks for @gcathelain for reviewing! I'll...

This is due to rounding errors that get introduced by converting forth and back from digital to analog signal space. EDF saves files in digital space (i.e. amplifier integers), and...

That's strange - do you have a sample file and code for me to check? (if you can't share the file publicly, you can upload it here: https://cloud.skjerns.de/index.php/s/Zc23TPbMrMBkK3, password='edfupload', files...

I think that's due to rounding errors in `edflib.c` https://github.com/holgern/pyedflib/blob/3bbcdc03396cf88d3bc73b276e65acf55fac520a/pyedflib/_extensions/c/edflib.c#L5725 ``` printf("My pre value is: %f\n", hdr->edfparam[i].phys_max); p = edflib_snprint_number_nonlocalized(str, hdr->edfparam[i].phys_max, 128); printf("My post value is: %s\n", str); # My...

I applied the patch created by Teuniz which adds a small offset to circumvent truncation. New version should be out soon.

whoops, might need to revert, patch has been retracted: https://gitlab.com/Teuniz/EDFlib/-/issues/15#note_1816944048

There is a fix in PR #159, if you install from that branch it should be fine. @holgern would you have time to have a look and merge PR? It...