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

feat(framework) Add grpc-adapter transport #3540

Merged
merged 52 commits into from
Jun 19, 2024
Merged

feat(framework) Add grpc-adapter transport #3540

merged 52 commits into from
Jun 19, 2024

Conversation

panh99
Copy link
Contributor

@panh99 panh99 commented May 31, 2024

This PR depends on #3536 and #3538

  • Added new transport type grpc-adapter
  • Amended run_client_app
  • Amended run_superlink

panh99 added 30 commits May 30, 2024 09:43
@panh99
add proto
e097ea4
@panh99
update test
2b3f44c
@panh99
fix test error
a69ee45
@panh99
Merge branch 'main' into protobuf-grpcadapter
c612c43
@panh99
add grpc adapter
58d823e
@panh99
complete grpc adapter class
88e69ce
@panh99
fix type
6783995
@panh99
Merge branch 'fix-metadata-type' into add-grpc-adapter-class
dd419f8
@panh99
fix check errors
6af6680
@panh99
Merge branch 'main' into add-grpc-adapter-class
53785de
@panh99
update doc string
f61b550
@panh99
add servicer
98cfda4
@panh99
make directories
05af29c
@panh99
fix date, update doc string
6c3ff4c
@panh99
Merge branch 'add-grpc-adapter-class' into test-grpcadapter
08581be
@panh99
Merge branch 'grpcadapter-servicer' into test-grpcadapter
da1a207
@panh99
amend run_client_app and run_superlink
4cd4a4f
@panh99
Merge branch 'main' into grpcadapter-servicer
083a6fb
@panh99
Merge branch 'main' into add-grpc-adapter-class
17cbcf3
@panh99
fix typing
e21aa1e
@panh99
Merge branch 'main' into add-grpc-adapter-class
ae3ca59
@panh99
Merge branch 'main' into grpcadapter-servicer
b3fb388
@panh99
Merge branch 'main' into test-grpcadapter
248a49f
@panh99
add arg for flower-superlink to specify grpc-adapter fleet api server…
ac09752 
… address
@panh99
parse arg
49aae70
@panh99
mv flwr-version-key to constant.py
c3d35bf
@panh99
Merge branch 'main' into grpcadapter-servicer
f1be661
@panh99
Merge branch 'main' into add-grpc-adapter-class
a477f1e
@panh99
mv constants to constant.py and handle control message
f4e0978
@panh99
Merge branch 'add-grpc-adapter-class' of https://github.com/adap/flower…
8877221 
… into add-grpc-adapter-class
panh99 added 21 commits June 10, 2024 10:02
@panh99
Merge branch 'add-grpc-adapter-class' into test-grpcadapter
95a076b
@panh99
merge main
0967d48
@panh99
Merge branch 'grpcadapter-servicer' into test-grpcadapter
2473004
@panh99
update
30c4a3e
@panh99
Merge branch 'main' into test-grpcadapter
b884048
@panh99
update log message and key name
ce2e4a5
@panh99
Merge branch 'main' into add-grpc-adapter-class
23b7507
@panh99
Merge branch 'add-grpc-adapter-class' into test-grpcadapter
df403cc
@panh99
Merge branch 'main' into grpcadapter-servicer
87bcaf0
@panh99
Merge branch 'main' into add-grpc-adapter-class
bd535e2
@panh99
Merge branch 'main' into add-grpc-adapter-class
3b2c101
@panh99
Merge branch 'main' into grpcadapter-servicer
b33c578
@panh99
update imports
e6be0cc
@panh99
update imports
0e1d737
@panh99
Merge branch 'main' into test-grpcadapter
0c354bb
@panh99
update supernode/app.py
8b2be0d
@panh99
update client/app.py and server/app.py
7ad13df
@panh99
Merge branch 'add-grpc-adapter-class' into test-grpcadapter
2a91a6b
@panh99
Merge branch 'grpcadapter-servicer' into test-grpcadapter
84dc775
@panh99
Merge branch 'main' into test-grpcadapter
1e75190
@panh99
restore
5c61be4
@panh99 panh99 marked this pull request as ready for review June 19, 2024 15:18
@danieljanes danieljanes enabled auto-merge (squash) June 19, 2024 17:21
@danieljanes danieljanes merged commit 88b08f4 into main Jun 19, 2024
29 checks passed
@danieljanes danieljanes deleted the test-grpcadapter branch June 19, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danieljanes danieljanes danieljanes approved these changes

@tanertopal tanertopal Awaiting requested review from tanertopal tanertopal 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.
2 participants
@panh99 @danieljanes