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

Windows SETUP #726

Merged
merged 4 commits into from
Apr 11, 2019
Merged

Windows SETUP #726

merged 4 commits into from
Apr 11, 2019

Conversation

loomlike
Copy link
Collaborator

@loomlike loomlike commented Apr 9, 2019

Description

Add windows setup instruction

Related Issues

#567, #491

Related bugs:
#698 (if you don't see this issue anymore, please close)
#709 This is a known issue: mmlspark lightgbm on Windows microsoft/SynapseML#483. I left a comment about how to reproduce the error.

Checklist:

  • My code follows the code style of this project, as detailed in our contribution guidelines.
  • I have added tests.
  • I have updated the documentation accordingly.

@roalexan
Copy link
Collaborator

roalexan commented Apr 9, 2019

'conda update anaconda' won't work on Windows DSVM. I need to run 'conda install anaconda'. Also, I think '-n root' is only used for Linux? Maybe we should have two sections like:

Linux

conda update conda -n root
conda update anaconda
Windows
conda update conda
conda install anaconda

SETUP.md Outdated Show resolved Hide resolved
SETUP.md Outdated Show resolved Hide resolved
SETUP.md Outdated Show resolved Hide resolved
SETUP.md Outdated Show resolved Hide resolved
@loomlike loomlike requested a review from nikhilrj April 10, 2019 17:47
@miguelgfierro
Copy link
Collaborator

merging since @roalexan approved

@miguelgfierro miguelgfierro merged commit 715322e into staging Apr 11, 2019
@miguelgfierro miguelgfierro deleted the loomlike/windows-setup branch April 11, 2019 13:14
yueguoguo pushed a commit that referenced this pull request Sep 9, 2019
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.

3 participants