Weirui Kuang comments

Results 58 comments of


                                            Weirui Kuang

About the Graph-DC Dataset

Thanks for your interests in FS. You can access the dataset in this branch: https://github.com/alibaba/FederatedScope/tree/feature/B-FHTL :)

Incorrect arguments to SHAWrapFedex init method

Thanks for the catching! The SHAWrapFedex will be deprecated in the future in #518 , but for now, adding `client_cfgs ` is the correct solution to avoid errors.

Client-side model parameters NOT updated during the FedEx training

Thanks for the suggestion, we noticed that if the cfg changes, the correct implementation is that the model (and the corresponding optimizer) should be re-instantiated. We will fix it ASAP.

TypeError: call_file_data() missing 1 required positional argument: 'client_cfgs'

`def call_file_data(config, client_cfgs):` should be right. How do you modify the function? `data_data_config = func(config)` should be `data_data_config = func(config, client_cfgs)`

TypeError: call_file_data() missing 1 required positional argument: 'client_cfgs'

> Another question, I can't install networkX, will it have any effect? If you are not planning to use the graph module, it won't have any effect.

Server global evaluation total number

Yes, you are right.

How can i get the output result?

You can get it from `trainer.ctx` or https://github.com/alibaba/FederatedScope/blob/480b67de851df2fa02b7cc2189f1803d947998d7/federatedscope/core/monitors/metric_calculator.py#L93

how can i use my saving model

You can use the following args in cfg file to load from ckpt. federate.restore_from | (string) '' | The checkpoint file to restore the model. | - -- | --...

how can i use my saving model

Duplicated #764

LDA splitter：ValueError: too many values to unpack (expected 2)

The data format should be list of tuples, such as `[(x, y), (x, y), (x, y), ...]`.