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

Updated ReadMe with Windows Installation #110

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gautamjajoo
Copy link
Member

Installation instructions for the development of the EvalAI-cli on Windows.
@RishabhJain2018 @vkartik97

@guyandtheworld
Copy link
Member

@gautamjajoo Did you test out all the commands of the CLI on the Windows machine? Does it have any shortcomings or bugs?

@gautamjajoo
Copy link
Member Author

@isht3 Yeah. I have checked all the commands. They work fine for me.

@gautamjajoo
Copy link
Member Author

@isht3 Please review!

Copy link
Member

@RishabhJain2018 RishabhJain2018 left a comment

Choose a reason for hiding this comment

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

Hi @gautamjajoo ,
Can you please change Development Setup -> Development Setup for Linux

@gautamjajoo
Copy link
Member Author

@RishabhJain2018 Updated!

@RishabhJain2018
Copy link
Member

@gautamjajoo Can you please update the instructions since now we support the feature of setting the token using evalai set_token <token> command.

@gautamjajoo
Copy link
Member Author

@RishabhJain2018 Done!

README.md Outdated
@@ -22,11 +22,54 @@ The goal of this package is to offer almost all the features available on the we

3. Login with your credentials and move to the profile section.

4. Click on `Get your Auth Token` to get the authentication token & download it as a JSON file.
4. To setup the auth token, run the following command:
Copy link
Member

Choose a reason for hiding this comment

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

This command will work only once evalai package is setup. Please move it after the step 4.

README.md Outdated

3. Login with your credentials and move to the profile section.

4. To setup the auth token, run the following command:
Copy link
Member

Choose a reason for hiding this comment

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

This command will work only once evalai package is setup. Please move it after the step 4.

README.md Show resolved Hide resolved

2. Now, Open web browser and hit the url http://127.0.0.1:8888.

3. Login with your credentials and move to the profile section.
Copy link
Member

Choose a reason for hiding this comment

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

Also, add instruction to copy the token from the profile section.

@gautamjajoo
Copy link
Member Author

@RishabhJain2018 Done the required changes!

@@ -45,7 +87,9 @@ git clone https://github.com/Cloud-CV/evalai-cli.git EvalAI-CLI
```bash
cd EvalAI-CLI
virtualenv -p python3 venv
source venv/bin/activate
cd venv/scripts
Copy link
Contributor

Choose a reason for hiding this comment

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

@gautamjajoo I think on Windows the 'S' in scripts is capital.
Also, how about the one-liner venv\Scripts\activate?
This is the recommended syntax. (reference)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants