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

Feature/SK-505 | Flush model update queues at new session + Buffer size config #476

Merged
merged 7 commits into from
Aug 2, 2023

Conversation

ahellander
Copy link
Member

@ahellander ahellander commented Jul 31, 2023

Description

Update the global round logic such that the combiner aggregation queues are flushed at the beginning of every new session. Make 'buffer size' configurable.

Resolves

  • SK-505
  • SK-520

@ahellander ahellander requested a review from Wrede July 31, 2023 07:27
@Wrede Wrede changed the title Feature/sk 505 Feature/SK-505 Jul 31, 2023
@Wrede Wrede changed the title Feature/SK-505 Feature/SK-505 | Flush model update queues at new session + Buffer size config Aug 1, 2023
fedn/fedn/network/combiner/server.py Outdated Show resolved Hide resolved
fedn/fedn/network/combiner/server.py Show resolved Hide resolved
fedn/fedn/network/combiner/round.py Show resolved Hide resolved
@ahellander ahellander requested a review from Wrede August 1, 2023 19:49
@ahellander ahellander merged commit a60d010 into develop Aug 2, 2023
15 checks passed
@ahellander ahellander deleted the feature/SK-505 branch August 2, 2023 13:48
Wrede added a commit that referenced this pull request Oct 25, 2023
…ze config (#476)

* model update queues gets flushed in the beginning of a new session

* Changed confusing log message

* buffer_size now configurable, solves sk-520

* exclude isort for protobuf files

* Deleted commented code

* Added response status message

---------

Co-authored-by: Andreas Hellander <andreas.hellander@gmail.com>
Co-authored-by: Fredrik Wrede <fredrik@scaleoutsystems.com>
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