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-972 | Suggested changes to quick start guide #691

Merged
merged 10 commits into from
Sep 3, 2024

Conversation

benjaminastrand
Copy link
Contributor

  • 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.

@github-actions github-actions bot added minor feature New feature or request docs and removed minor labels Sep 2, 2024
Copy link
Member

@ahellander ahellander left a 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.

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.
Copy link
Member

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?

Copy link
Contributor Author

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.

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'.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same question here.

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.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same question.

@github-actions github-actions bot added the minor label Sep 3, 2024
@ahellander ahellander self-requested a review September 3, 2024 08:24
@ahellander ahellander merged commit 1ee8aca into master Sep 3, 2024
9 checks passed
@ahellander ahellander deleted the feature/SK-972 branch September 3, 2024 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs feature New feature or request minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants