Replaceing KV actor by Ray API

ray-project / rayfed

A multiple parties joint, distributed execution engine based on Ray, to help build your own federated learning frameworks in minutes.

https://rayfed.readthedocs.io

Apache License 2.0

91 stars 20 forks source link

Replaceing KV actor by Ray API #136

Closed NKcqx closed 8 months ago

NKcqx commented 1 year ago

Resolves #135

NKcqx commented 1 year ago

Unfortunately, Ray's client mode internal_kv accessing has a BUG that causes the call of internal_kv_del takes no effect. This PR can't be merged untill this issue is fixed.

NKcqx commented 1 year ago

It turns out to be a dumb BUG in Ray, fix PR: https://github.com/ray-project/ray/pull/37031/files

jovany-wang commented 1 year ago

If the current ray doesn't support it, how can we handle to get compatible with all versions?

jovany-wang commented 8 months ago

Do we need go with this PR? @ray-project/rayfed