Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix: add concrete ml tutorial on fedbiomed website #1120

Closed
wants to merge 1 commit into from

Conversation

ybouilla
Copy link
Contributor

PR description

Adds missing link to concrete-ml tutorial on website

Please do not delete this branch, since it is a hotfix (we will merge this branch in develop as well)

Developer Certificate Of Origin (DCO)

By opening this merge request, you agree to 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 PR 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_breakpoint/load_state_breakpoint in aggregators, strategies, secagg, etc.)
  • if modified a component with versioning (config files, breakpoint, messaging protocol) then update the version following the rules in common/utils/_versions.py

Adds missing link to `mkdocs.yml` file, in order to display `concrete-ml` tutorial on
Fed-BioMed website
@mvesin
Copy link
Member

mvesin commented May 30, 2024

hi @ybouilla this PR does not seem necessary.

There is already a menu named End-to-end Privacy Preserving Training and Inference on Medical Data refering to the concrete ML tuto in the documentation, isn't it ?

@ybouilla
Copy link
Contributor Author

Yes you are right... I missed this entry in the mkdocs. Closing since it is irrelevant

@ybouilla ybouilla closed this May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants