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

Adding code that showcases federated learning with Covalent #1788

Merged
merged 17 commits into from
Oct 3, 2023

Conversation

FilipBolt
Copy link
Contributor

@FilipBolt FilipBolt commented Sep 25, 2023

Adding code that showcases federated learning with Covalent, not in tutorial mode, but only to link to it from the blog.

  • I have added the tests to cover my changes.
  • I have updated the documentation and CHANGELOG accordingly.
  • I have read the CONTRIBUTING document.

@FilipBolt FilipBolt requested review from a team and santoshkumarradha as code owners September 25, 2023 17:06
@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

Merging #1788 (b60df8d) into develop (7967f16) will increase coverage by 1.91%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1788      +/-   ##
===========================================
+ Coverage    78.17%   80.08%   +1.91%     
===========================================
  Files          116      231     +115     
  Lines         6959    10179    +3220     
  Branches         0      193     +193     
===========================================
+ Hits          5440     8152    +2712     
- Misses        1519     1894     +375     
- Partials         0      133     +133     
Flag Coverage Δ
Dispatcher 86.37% <ø> (ø)
Functional_Tests 51.97% <ø> (?)
SDK 74.56% <ø> (ø)
UI_Backend 88.57% <ø> (?)
UI_Frontend 73.43% <ø> (?)

FilipBolt and others added 8 commits September 26, 2023 16:38
…iqHQ#1710)

* Bump semver from 7.3.8 to 7.5.2 in /.github/actions/changelog

Bumps [semver](https://github.com/npm/node-semver) from 7.3.8 to 7.5.2.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v7.3.8...v7.5.2)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

* changelog

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will Cunningham <[email protected]>
…#1732)

* Bump word-wrap from 1.2.3 to 1.2.4 in /covalent_ui/webapp

Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updated changelog

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Prasy12 <[email protected]>
…qHQ#1722)

* Bump tough-cookie from 4.0.0 to 4.1.3 in /covalent_ui/webapp

Bumps [tough-cookie](https://github.com/salesforce/tough-cookie) from 4.0.0 to 4.1.3.
- [Release notes](https://github.com/salesforce/tough-cookie/releases)
- [Changelog](https://github.com/salesforce/tough-cookie/blob/master/CHANGELOG.md)
- [Commits](salesforce/tough-cookie@v4.0.0...v4.1.3)

---
updated-dependencies:
- dependency-name: tough-cookie
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updated changelog

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Prasy12 <[email protected]>
* Bump semver from 5.7.1 to 5.7.2 in /covalent_ui/webapp

Bumps [semver](https://github.com/npm/node-semver) from 5.7.1 to 5.7.2.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/v5.7.2/CHANGELOG.md)
- [Commits](npm/node-semver@v5.7.1...v5.7.2)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updated changelog

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Prasy12 <[email protected]>
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 23.7.0 → 23.9.1](psf/black@23.7.0...23.9.1)
- [github.com/pre-commit/mirrors-prettier: v3.0.1 → v3.0.3](pre-commit/mirrors-prettier@v3.0.1...v3.0.3)
- [github.com/hadialqattan/pycln: v2.2.1 → v2.2.2](hadialqattan/pycln@v2.2.1...v2.2.2)

* Updated changelog

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sankalp Sanand <[email protected]>
@FilipBolt FilipBolt requested a review from a team as a code owner October 2, 2023 17:59
Copy link
Contributor

@araghukas araghukas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@FilipBolt looks good! Requesting just one change to a small cell in the intro.

doc/source/tutorials/federated_learning/source.ipynb Outdated Show resolved Hide resolved
@araghukas araghukas merged commit 6885bb2 into AgnostiqHQ:develop Oct 3, 2023
12 checks passed
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.

4 participants