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
Update quickstart-pytorch example to use app
model
#3116
Conversation
…ower into tabdar/update-quickstart-pytorch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @tabdar-khan , I added some small changes to the code that were needed to keep things consistent w/ and w/o Flower Next. Let me know what you think
Or, to try the workflow example, run: | ||
|
||
```bash | ||
flower-server-app server_workflow:app --insecure | ||
``` | ||
|
||
Or, to try the custom server function example, run: | ||
|
||
```bash | ||
flower-server-app server_custom:app --insecure | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was only relevant for the exampes/app-pytorch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah! I see.
______________________________________________________________________ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm thinking we could remove these, since the immediately below section title will introduce one already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure.
@@ -72,3 +74,29 @@ python3 client.py --partition-id 1 | |||
``` | |||
|
|||
You will see that PyTorch is starting a federated training. Look at the [code](https://github.com/adap/flower/tree/main/examples/quickstart-pytorch) for a detailed explanation. | |||
|
|||
______________________________________________________________________ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove? (also those in #3117?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. I don't have a strong opinion about this
Issue
quickstart-pytorch example is outdated as it is not using the new app model from Flower Next.
Description
client.py
andserver.py
files in the quickstart-pytorch example to use the newapp
model.Changelog entry
Update quickstart-pytorch example to use
app
model