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

Update for mirai clusters #5

Closed shikokuchuo closed 5 months ago

shikokuchuo commented 9 months ago

@HenrikBengtsson I’ve prepared this PR to replace the previous proof of concept method with actual ‘miraiCluster’.

I think it is now ready to be fully fleshed out for remote connections etc.

What would be useful is some indication on how you think things should be done: at a high level, whether to keep the implementation within the framework of ‘parallel’ clusters (perhaps more convenient / compatible), or simply look to map the mirai / future interfaces more directly (perhaps allowing more flexibility / functionality).

Once you've thought about the above perhaps we can set a direction.

Happy to help get this package across the line.

shikokuchuo commented 9 months ago

I am not sure why the "future_tests" are failing on CI. R CMD check passes on CI as locally - I thought those just called the relevant future.tests functions.

shikokuchuo commented 8 months ago

My changes are not quite complete, need to add more commits. Not ready to merge yet.

shikokuchuo commented 5 months ago

This was a somewhat abortive early attempt, already superseded by your current updates. I see you are making great progress!