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

Remove compute2 dependency from the scatter-gather code #295

Merged
merged 71 commits into from
Mar 9, 2023

Conversation

garg-amit
Copy link
Contributor

Purpose

  • Remove compute2 dependency from the scatter-gather code as we provision only 1 compute/silo by default.

What is the expected review turnaround time?

Urgency:

  • High (needs review today)
  • Medium (needs review within a few days - most common)
  • Low (can wait a week)

Does this introduce a breaking change?

[x] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Jeff Omhover and others added 30 commits August 15, 2022 23:03
SDK v2 version of the FL pipeline submission script
* docker file stub

* move docker file, implement feedback

* login before setting subscription

* login before setting subscription

* use default k8s version

* pin latest version since default won't work

* remove executionpolicy part, other small updates

* clarify to change job file _in docker filesystem_

* login before setting subscription

* formatting

* \ -> /

* install azureml-core in docker file

* propagate changes to section 7

* fix dataset creation command

Co-authored-by: thomasp-ms <XXX@me.com>

Co-authored-by: thomasp-ms <XXX@me.com>
garg-amit and others added 26 commits September 2, 2022 17:07
Federated learning real experiment - MNIST dataset
Documentation to run an FL experiment
@garg-amit garg-amit marked this pull request as ready for review March 9, 2023 23:38
@garg-amit garg-amit merged commit 68eadc9 into main Mar 9, 2023
@garg-amit garg-amit deleted the gargamit/fix-appinsight-error branch March 9, 2023 23:40
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

3 participants