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

REST API walk-through #964

Merged
merged 1 commit into from
Aug 19, 2016
Merged

Conversation

gheinrich
Copy link
Contributor

No description provided.


### Creating the classification dataset

In order to create a dataset, you will first need to log in.
Copy link
Member

Choose a reason for hiding this comment

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

It's probably worth noting that you don't need to do this with a cookie. You can just add -F username=luke to the end of any curl command.

https://github.com/NVIDIA/DIGITS/blob/v4.0.0/digits/utils/auth.py#L11-L13

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I have added a note about that.

@lukeyeager
Copy link
Member

Do you think this file might make more sense under /docs/ rather than under /examples/?

Either way, we should add a link to it at the end of /docs/GettingStarted.md.

@gheinrich
Copy link
Contributor Author

I have moved the file to /docs and added the link at the end of /docs/GettingStarted.md.

@gheinrich gheinrich force-pushed the dev/rest-api-tutorial branch 2 times, most recently from 94bd053 to eeb4f20 Compare August 19, 2016 17:11
@lukeyeager
Copy link
Member

Thanks! LGTM

@lukeyeager lukeyeager merged commit ee72013 into NVIDIA:master Aug 19, 2016
@gheinrich gheinrich deleted the dev/rest-api-tutorial branch August 19, 2016 18:42
SlipknotTN pushed a commit to cynnyx/DIGITS that referenced this pull request Mar 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants