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

Update install doc #696

Merged
merged 4 commits into from
Nov 2, 2016
Merged

Update install doc #696

merged 4 commits into from
Nov 2, 2016

Conversation

kohr-h
Copy link
Member

@kohr-h kohr-h commented Nov 1, 2016

Some updates after the recent installation session. I also extended the odlcuda install section with instructions to use the conda build recipe. That should in fact be the most useful variant for most users.

I'll also update the ODL conda package so it doesn't pull in the nomkl package on Windows.

To do this, open Spyder and use the navigation bar to open "Tools -> Preferences".
Click on "Python interpreter" and change the first setting "Select the Python interpreter for all Spyder consoles" from the default setting to "Use the following Python interpreter:".
In the text field, fill in the path to the Python executable in your newly created conda environment.
For example, if you installed Miniconda (or Anaconda) in ``C:\Programs\Miniconda3``, then the environment's Python interpreter is ``C:\Programs\Miniconda3\envs\odl-py35\bin\python.exe``.
Copy link
Member Author

Choose a reason for hiding this comment

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

Is this correct?

Copy link
Member

Choose a reason for hiding this comment

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

Frankly I dont know, I need to set up a fresh VM to try this out.

Copy link
Member Author

Choose a reason for hiding this comment

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

Don't worry, I'll ask Joost tomorrow.

Copy link
Member Author

Choose a reason for hiding this comment

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

Tested now, works after some tweaks to the conda packages. It ain't pretty...

@@ -194,7 +206,7 @@ TL;DR
Installing a Python interpreter
===============================
Open a terminal and type ``python`` + Enter.
If a Python prompt appears, you already have an interpreter installed and can skip this step (exit by pressing Ctrl+D).
If a Python prompt appears, you already have an interpreter installed and can skip this step (exit by pressing Ctrl+D or running ``exit()``).
Copy link
Member

Choose a reason for hiding this comment

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

dislike "a or b" for beginner level instructions, simply give them one option (here, id go for exit())

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, I added it because Ctrl+D doesn't work on Windows.


.. code-block:: bash

$ conda install numpy scipy future [matplotlib]
Copy link
Member

Choose a reason for hiding this comment

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

Is this verified?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, that's what I did last Tuesday on Joost's laptop.

@adler-j
Copy link
Member

adler-j commented Nov 1, 2016

Great improvements overall, simply need to check that the windows instructions actually work.

@kohr-h
Copy link
Member Author

kohr-h commented Nov 1, 2016

Great improvements overall, simply need to check that the windows instructions actually work.

Thanks, I'll check tomorrow. Perhaps we can even dare to do the "Frank test" again :-)

@kohr-h kohr-h force-pushed the update_install_doc branch from 0b66ec0 to 04ab147 Compare November 1, 2016 22:56
@kohr-h kohr-h force-pushed the update_install_doc branch from 04ab147 to 0332423 Compare November 1, 2016 23:10
@adler-j
Copy link
Member

adler-j commented Nov 2, 2016

Go ahead after that!

@kohr-h kohr-h mentioned this pull request Nov 2, 2016
@kohr-h kohr-h force-pushed the update_install_doc branch 2 times, most recently from 80dd46c to 9e2c2d1 Compare November 2, 2016 15:39
@kohr-h kohr-h force-pushed the update_install_doc branch 2 times, most recently from 08e86d4 to bdf2ffe Compare November 2, 2016 17:04
@kohr-h kohr-h force-pushed the update_install_doc branch from bdf2ffe to 0793e14 Compare November 2, 2016 17:21
@kohr-h kohr-h merged commit 08f568d into master Nov 2, 2016
@kohr-h kohr-h deleted the update_install_doc branch November 2, 2016 17:32
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.

2 participants