Shaked Zychlinski

Results 14 comments of Shaked Zychlinski

I see what you did here, nice idea. I got COVID a few days ago, so I'm a little less focused now, I'll take a deeper look once I'm better....

`test_datetime_data` fails, and I don't understand why..

hey @matbb - I just merged a PR that enforces using Black. This branch needs to merge `master` as there are conflicts

So this PR needs a whole new refactor due to the new parallelizing mechanism that was added. I tried to this, but broke the whole thing..

Fixed. Need to add new test for the new option that was added here.

Hey @KonradHoeffner - parallelizing would be a great idea, and contributions would be more than welcomed!

Hey @KonradHoeffner - please open a bug report and add all this there, so we won't make different issues here. Anyway, the best way to run and test the version...

Hey @ThomasBury - first of all, thanks for your contribution :) I'm wondering here if @mahieyin-rahmun thumbs-up means he's into your code too :) I think adding your parallelization of...

@ThomasBury & @KonradHoeffner - no worries and no pressure :) whenever you have the time, simply open a new PR.