Michael Straube

Results 9 comments of Michael Straube

Tested and it works, thanks.

To me this looks like a bug in the [_round_float_to_8_characters](https://github.com/the-siesta-group/edfio/blob/main/edfio/edf_signal.py#L40) method from edfio. For `-2006585738.63211`, which is the value of pmin in the provided example data, it returns `-2006590000.0`. Script...

Looking at the physical_range of the signals using edfio directly, it seems that `-2006585738.63211` is wrong in the first place. ```python #!/usr/bin/env python3 import edfio edf = edfio.read_edf("SC4001E0-PSG.edf") for sig...

I think the issue is with the .edf file itself. The header does not specify an unit for the "Resp oro-nasal" channel and physical range min -2048, but the actual...

I did a `git bisect` using the script provided under `Steps to reproduce` and it confirmed that #12536 introduced the issue.

> [@mistraube](https://github.com/mistraube) are you up for trying to fix it? You seem to be on a roll this week :) Yes, I looked into it and I think the problem...

> If we are in such a case and `noverlap` is greater than the `span length` we get the error. > actually it's `greater or equal` here

> I have made a final push to the branch using a fixed, high z-order (50) for the left axis spine, as the dynamic z-order (i.e. solution of @mistraube )...

@mohit-malpote are you still interested in working on this?