-
Notifications
You must be signed in to change notification settings - Fork 36
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-972 | Suggested changes to quick start guide #691
Conversation
benjaminastrand
commented
Sep 2, 2024
- Explain that git clone is only necessary to get the example code if it wasn't already used in the previous step.
- Changed some step-by-step instructions to numbered lists instead of 'löpande' text.
- Clarify that sessions shouldn't be started immediately after uploading package and seed files.
- Clarify how to select a dataset partition for a client when creating several clients.
- Changed the model overview image to include only 5 rounds, as default settings are recommended in the tutorial.
- Fixed minor typos.
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.
Looks great! See comment/question.
docs/quickstart.rst
Outdated
In the Studio UI, navigate to the project you created in step one and click on the "Sessions" tab. Click on the "New Session" button. Under the "Compute package" tab, select a name and upload the generated package file. Under the "Seed model" tab, upload the generated seed file: | ||
The next step is to initialize the server side with the client code and the initial global model. In the Studio UI, | ||
|
||
1. Navigate to the project you created in step 1 and click on the "Sessions" tab. |
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.
Will this format as an itemized list when built with sphinx?
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.
Yes. However, I changed it to the recommended syntax using '#.' for ordered lists.
docs/quickstart.rst
Outdated
Go to the Clients' tab and click 'Connect client'. Download a client configuration file and save it to the root of the ``examples/mnist-pytorch folder``. | ||
Rename the file to 'client.yaml'. | ||
|
||
1. Go to the 'Clients' tab and click 'Connect client'. |
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.
Same question here.
docs/quickstart.rst
Outdated
In Studio click on the "Sessions" link, then the "New session" button in the upper right corner. Click the "Start session" tab and enter your desirable settings (the default settings are good for this example) and hit the "Start run" button. | ||
In Studio, | ||
|
||
1. Click on the "Sessions" link, then the "New session" button in the upper right corner. |
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.
Same question.