Mentions légales du service

Skip to content

provide default demographics_transform if None given by researcher

MR description

Closes #429 (closed).

The suggested fix is to provide a default empty transform when the researcher did not provide one themselves. This allows researchers that only use demographics for filtering data to do so without additional (counterintuitive) effort.

On the other hand, this means that if someone had the expectation that they would be able to use their demographics data without specifying a transform, they are going to be disappointed. Hence I updated the comments in the notebook, and mirrored those changes in the docs (see fedbiomed.gitlabpages.inria.fr!70).

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.)
Edited by CREMONESI Francesco

Merge request reports