Definitely looks more like the Julia one now…

This is making me wish Python had fake-able timers like Jest, so we could write a nice test for this! 😅

This is making me wish Python had fake-able timers like Jest, so we could write a nice test for this! 😅

yeah agree that would be indeed nice 😄

It looks like there is https://docs.python.org/3/library/unittest.mock.html

Maybe you could try using that to write a small unit test for poll_with_specified_overhead, like we did for the profiler poller? Since this logic is important for performance and has caused bugs in the past, it'd be nice to have it under test.

@vilterp it actually had couple tests, now I have added more tests

Hm, looks like tests failing with

Exception: max tries 1 exhausted

@vilterp all the tests are passed. Where do you see it?

@vilterp
it was failing with Exception: max tries 1 exhausted here: https://github.com/RelationalAI/rai-sdk-python/pull/151/files#diff-f4c8f93a330162e591e147d72a2dba80b81510662e5608f2d130bf63d5220ed7R33 However, I already fixed it:
I increased max_tries so that the function doesn't exit on the first iteration. Then sse a try except block to catch the exception but still check the behavior of time.sleep, because the goal of this test is to test initial delay.

Oh ok, I guess the github UI didn't refresh itself and I was looking at an out of date run. Will review again on Mon