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

Python installation instructions #651

Merged
merged 3 commits into from
Feb 10, 2021
Merged

Python installation instructions #651

merged 3 commits into from
Feb 10, 2021

Conversation

adswa
Copy link
Contributor

@adswa adswa commented Feb 9, 2021

Would you mind reviewing this, @Remi-Gau? I tried to add installation instructions for Python 3 based on your pointers and thoughts in #649. :)

…hon instructions

Thanks to @Remi-Gau for the suggestion and the OHBM hackathon team for the original instructions

This adds mostly OS-independent installation instructions for Python, together with a more in-depth
highlighting of the need for (and differences) of Python 3.
It also fixes issue #638 by explicitly warning against the Windows Store version of Python, as this
at the moment does not configure the /home/adina/env/handbook2/bin:/home/adina/.dotfiles/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/X11R6/bin:/usr/local/games:/usr/games:/home/adina/.local/bin:/home/adina/bin:/snap/bin appropriately.

Fixes #649.
Copy link
Contributor

@Remi-Gau Remi-Gau left a comment

Choose a reason for hiding this comment

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

This looks good to me. Asked some of my lab to cross check (if they have the time).

One thing that could be worth mentioning is that it is not because they have installed Python 3 that people can / should uninstall Python 2 (unless they want to break things on their system that still relies on Python 2). I have been asked that question a couple of times already, so I suspect others will ask this question too. What do you think?

@adswa
Copy link
Contributor Author

adswa commented Feb 10, 2021

One thing that could be worth mentioning is that it is not because they have installed Python 3 that people can / should uninstall Python 2 (unless they want to break things on their system that still relies on Python 2). I have been asked that question a couple of times already, so I suspect others will ask this question too. What do you think?

I think it makes a lot of sense mentioning this! Will add this, thanks :)

@adswa
Copy link
Contributor Author

adswa commented Feb 10, 2021

@all-contributors please add @Remi-Gau for review

@allcontributors
Copy link
Contributor

@adswa

I've put up a pull request to add @Remi-Gau! 🎉

Copy link
Contributor

@Remi-Gau Remi-Gau left a comment

Choose a reason for hiding this comment

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

Good to merge for me.

Thanks @adswa !!!

Keep it installed, especially if you are using Linux or MacOS.
Python 2 existed for 20 years and numerous software has been written in it.
It is quite likely that some basic operating system components or legacy software on your computer is depending on it, and uninstalling a preinstalled Python 2 from your system will likely render it unusable.
Install Python 3, and have both versions coexist peacefully.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Install Python 3, and have both versions coexist peacefully.
Install Python 3, and have both versions coexist peacefully.

hahaha

Did you mean to write:

... coexist peacefully (as long as you use some sort of virtual environment, otherwise things could get a lot less peaceful very quickly).

😉

Just kidding. This is perfect.

@adswa
Copy link
Contributor Author

adswa commented Feb 10, 2021

Great, thx!

@adswa adswa merged commit da4a555 into master Feb 10, 2021
@adswa adswa deleted the pythoninstall branch February 10, 2021 09:53
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