search

futureverse / future.mirai

:rocket: R package future.mirai: A Future API for Parallel Processing using 'mirai'
https://future.mirai.futureverse.org/
21 stars 1 forks source link

mirai interface updates #3

Closed shikokuchuo closed 1 year ago

shikokuchuo commented 1 year ago

Great to finally see this Henrik!

Perhaps open discussions for easier discussion?

I have not had the chance to really look over yet, but should mention that the latest version of mirai has introduced a new dedicated function status() and deprecated the use of daemons() with no arguments.

Should I open PRs in case there are other things I notice, or should I give you some time to catch up first?

HenrikBengtsson commented 1 year ago

Hi. Thanks for letting me know. Yes, I'd be happy for a PR if you've got the cycles.

shikokuchuo commented 1 year ago

I'll have to find time to look over the package in full, so I've opened a PR just for the obvious changes. I will open new issues for other comments.

HenrikBengtsson commented 1 year ago

Thank you for this. Sorry for the latency. Merged.