The solution for _clock_sync does not work

[agchitu]

The solution as it is presented here:

pyxdf/src/pyxdf/pyxdf.py

Line 858 in 63f2c3d

np.argmin(

This does not manage to properly identify the time stamp ranges.
The reason is that using np.argmin([True] * len(time_stamps)) will return 0 which is not correct. It should return len(time_stamps).

This could be a working solution: next((i for i, v in enumerate(lst) if not v), len(lst)) where lst = the np.less(...)

[cbrnr]

Good catch! Can you open a PR with the proposed change?

[sappelhoff]

cc @mscheltienne

[mscheltienne]

Thanks for the cc, that seems like a good fix :)

[sappelhoff]

@cbrnr could we get a patch release after this fix is merged, please?