ValueError when time is floating point in _select_lp

[ozzdemir]

Hi, when I gave floating point time instances, random.randint() gives Value error. Is there a particular reason for using random.randint() instead of random.uniform() ? [I locally replaced it with random.uniform(), and it worked with floating time also].

modest-py/modestpy/estimation.py

Line 612 in d539a94

new_t0 = random.randint(t0, tend - lp_len)

Note that I did not give lp_frame explicitly, so it took the time from "ideal". (If the choice is related to performance, maybe rounding can be made?)

Thanks in advance

[krzysztofarendt]

Hi, this is probably a bug. Could you make a PR with a fix including a simple unit test for this case?
https://jarv.is/notes/how-to-pull-request-fork-github/

If not, I will try to find some time to do it, but I cannot promise any deadline. I have limited time for maintaining this project.