Mentions légales du service

Skip to content

Avoid MP-SPDZ configuration file conflicts in all cases

VESIN Marc requested to merge feature/485-mpspdz-file-conflicts into develop

MR description

This MR adresses #485 (closed).

You can test with:


# clean configuration
source ./scripts/fedbiomed_environment clean
./scripts/fedbiomed_run  network

# create, configure, launch 2 nodes + 1 researcher
./scripts/fedbiomed_run node config config-n1.ini configuration create
./scripts/fedbiomed_run node config config-n1.ini -am /data/mvesin/data
./scripts/fedbiomed_run node config config-n1.ini start
./scripts/fedbiomed_run node config config-n2.ini configuration create
./scripts/fedbiomed_run node config config-n2.ini -am /data/mvesin/data
./scripts/fedbiomed_run node config config-n2.ini start
./scripts/fedbiomed_run researcher configuration create
./scripts/fedbiomed_run researcher start

# initialize certificates for the 3 components
./scripts/fedbiomed_configure_secagg  node   # or researcher
./scripts/fedbiomed_run certificate-dev-setup

# then test with `notebooks/general-tutorial-secagg.ipynb`

Developer Certificate Of Origin (DCO)

By opening this merge request, you agree the Developer Certificate of Origin (DCO)

This DCO essentially means that:

  • you offer the changes under the same license agreement as the project, and
  • you have the right to do that,
  • you did not steal somebody else’s work.

License

Project code files should begin with these comment lines to help trace their origin:

# This file is originally part of Fed-BioMed
# SPDX-License-Identifier: Apache-2.0

Code files can be reused from another project with a compatible non-contaminating license. They shall retain the original license and copyright mentions. The CREDIT.md file and credit/ directory shall be completed and updated accordingly.

Guidelines for MR review

General:

Specific to some cases:

  • update all conda envs consistently (development and vpn, Linux and MacOS)
  • if modified researcher (eg new attributes in classes) check if breakpoint needs update (breakpoint/load_breakpoint in Experiment(), save_state/load_state in aggregators, strategies, secagg, etc.)

Merge request reports