8000 misc: config update on async mnist example by myungjin · Pull Request #364 · cisco-open/flame · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

misc: config update on async mnist example #364

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

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

myungjin
Copy link
Contributor
@myungjin myungjin commented Mar 8, 2023

Description

The config for async mnist example is updated so that the broker format is identical across other examples. It basically allows switching between local mqtt broker and local metaserver.

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@myungjin myungjin requested a review from lkurija1 March 8, 2023 20:44
@codecov-commenter
Copy link
codecov-commenter commented Mar 8, 2023

Codecov Report

Merging #364 (fc96d99) into main (3f08b7f) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #364   +/-   ##
=======================================
  Coverage   14.39%   14.39%           
=======================================
  Files          48       48           
  Lines        2771     2771           
=======================================
  Hits          399      399           
  Misses       2354     2354           
  Partials       18       18           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

The config for async mnist example is updated so that the broker
format is identical across other examples. It basically allows
switching between local mqtt broker and local metaserver.
@myungjin myungjin force-pushed the async_mnist_config branch from 84718f2 to fc96d99 Compare March 10, 2023 15:03
@myungjin myungjin merged commit 12f325e into cisco-open:main Mar 14, 2023
@myungjin myungjin deleted the async_mnist_config branch March 14, 2023 15:54
alexandruuBytex pushed a commit to alexandruuBytex/flame that referenced this pull request Mar 15, 2023
* Remove the build-image command

* Configure golangci.yml to add the linter exceptions

* Revert changes from .golangci.yml

---------

Co-authored-by: Alex Ungurean <aungurea@cisco.com>

Move dataSpec and modelSpec to separate json files

Remove aggregators and trainers

Remove aggregators and trainers

Modify the test to respect the new structure

misc: config update on async mnist example (cisco-open#364)

The config for async mnist example is updated so that the broker
format is identical across other examples. It basically allows
switching between local mqtt broker and local metaserver.
alexandruuBytex pushed a commit to alexandruuBytex/flame that referenced this pull request Mar 16, 2023
* Remove the build-image command

* Configure golangci.yml to add the linter exceptions

* Revert changes from .golangci.yml

---------

Co-authored-by: Alex Ungurean <aungurea@cisco.com>

Move dataSpec and modelSpec to separate json files

Remove aggregators and trainers

Remove aggregators and trainers

Modify the test to respect the new structure

misc: config update on async mnist example (cisco-open#364)

The config for async mnist example is updated so that the broker
format is identical across other examples. It basically allows
switching between local mqtt broker and local metaserver.

Remove unwanted changes
dhruvsgarg pushed a commit to dhruvsgarg/flame that referenced this pull request Oct 18, 2024
The config for async mnist example is updated so that the broker
format is identical across other examples. It basically allows
switching between local mqtt broker and local metaserver.
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.

3 participants
0