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.

Sign up for GitHub

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

Jump to bottom

Add tensorboard mod example #2723

Closed
wants to merge 134 commits into from
Closed

Add tensorboard mod example #2723

wants to merge 134 commits into from

Conversation

charlesbvll
Copy link
Member

Issue

Description

Related issues/PRs

Proposal

Explanation

Checklist

  • Implement proposed change
  • Write tests
  • Update documentation
  • Update changelog
  • Make CI checks pass
  • Ping maintainers on Slack (channel #contributions)

Any other comments?

danieljanes and others added 30 commits September 19, 2023 20:25
@danieljanes
Decouple client and app
4886c77
@danieljanes
Merge branch 'main' into decouple-client
fb8ba3d
@danieljanes
Merge branch 'main' into decouple-client
fca2187
@danieljanes
Merge branch 'main' into decouple-client
5e5fec6
@danieljanes
Merge branch 'main' into decouple-client
c3e82b7
@danieljanes
Shorten line
1223f90
@danieljanes
Merge branch 'main' into decouple-client
555d89a
@danieljanes
Add comments to main client loop
a7c57d6
@danieljanes
Refactor start_client
9a20a0c
@danieljanes
Merge branch 'main' into decouple-client
d65601b
@danieljanes
Merge branch 'refactor-start-client' into decouple-client
82df0a3
@danieljanes
Format code
32e4973
@danieljanes
Add type hint
11bf509
@danieljanes
Merge branch 'refactor-start-client' into decouple-client
d097096
@charlesbvll
Test with Iterator
a1a647a
@charlesbvll
Working solution (I think)
ce2bbb8
@charlesbvll
Fix imports
7617335
@danieljanes
Merge branch 'refactor-start-client' into decouple-client
d9a7318
@danieljanes
Merge branch 'main' into decouple-client
46acb9d
@danieljanes
Merge branch 'main' into decouple-client
e61c91a
@danieljanes
Move task execution into Flower app
156aad0
@danieljanes
Enable --app in flower-client
a47a760
@danieljanes
Merge branch 'main' into decouple-client
77bfc04
@danieljanes
Add comments to start_client
90058f9
@danieljanes
Handle control messages in a separate function
86c1473
@danieljanes
Change comment
9bc1349
@danieljanes
Merge branch 'refactor-start-client-comments' into refactor-start-cli…
ee87393 
…ent-control-message
@danieljanes
Update comments
1691340
@danieljanes
Update comments
7d732e9
@danieljanes
Format
9f6472f
panh99 and others added 19 commits December 1, 2023 16:59
@panh99
rename make_app to make_fc
2df4116
@panh99
fix typo
2d1b983
@panh99
update naming
d897fe8
@panh99
Merge branch 'main' into middleware
b8f25cd
@panh99
update tutorial
c8fed7f
@panh99
Merge branch 'middleware' of https://github.com/adap/flower into midd…
0f15a30 
…leware
@panh99
fix format
dba7c42
@panh99
Merge branch 'main' into middleware
8daac9b
@charlesbvll
First draft
dd294de
@charlesbvll
Merge branch 'main' of https://github.com/adap/flower into add-tensor…
3cdca07 
…board-middleware
@charlesbvll
Remove outdated file
fda20f5
@charlesbvll
Better naming
3218dfd
@charlesbvll
Reorganize imports
bae9d51
@charlesbvll
Reorganize imports the correct way
3711e0d
@tanertopal
Merge branch 'main' into add-tensorboard-middleware
af835c9
@tanertopal
Merge branch 'main' into add-tensorboard-middleware
73d41c0
@charlesbvll
Merge branch 'main' of https://github.com/adap/flower into add-tensor…
dea309d 
…board-middleware
@charlesbvll
Update mod
17c2061
@charlesbvll
Delete outdated file
5cd14fc
@charlesbvll charlesbvll changed the title Add tensorboard middleware Add tensorboard mod example Feb 12, 2024
@adap adap deleted a comment from notion-workspace bot Feb 20, 2024
@adap adap deleted a comment from notion-workspace bot Feb 20, 2024
@charlesbvll charlesbvll closed this Mar 7, 2024
@charlesbvll charlesbvll deleted the add-tensorboard-middleware branch March 7, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chongshenng chongshenng Awaiting requested review from chongshenng chongshenng will be requested when the pull request is marked ready for review chongshenng is a code owner

@jafermarq jafermarq Awaiting requested review from jafermarq jafermarq will be requested when the pull request is marked ready for review jafermarq is a code owner

@tanertopal tanertopal Awaiting requested review from tanertopal tanertopal will be requested when the pull request is marked ready for review tanertopal is a code owner

@danieljanes danieljanes Awaiting requested review from danieljanes danieljanes will be requested when the pull request is marked ready for review danieljanes is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@charlesbvll @danieljanes @panh99 @tanertopal