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
92 stars 21 forks source link

Fix missing messages_max_size_in_bytes config #182

Closed NKcqx closed 10 months ago

NKcqx commented 11 months ago

The messages_max_size_in_bytes is a config item in CrossSiloMessageConfig. The original design is that when users customize their proxy actor and config, this field should be converted to the corresponding field. E.g. when using grpc proxy, this field should be equivalent to setting "grpc.max_send_message_length" and "grpc.max_receive_message_length" in GrpcCrossSiloMessageConfig.grpc_channel_options. However, this field get missed when converting to GrpcCrossSiloMessageConfig.

This PR fix this bug and should solving #181 and this issue: https://github.com/secretflow/secretflow/issues/952