CAM-Gerlach
commented
5 years ago I recommend removing that sentence entirely as its essentially redundant, and just embedding the link directly in the previous sentence, like so (with some other corrections/updates):
Although there are multiple ways to meet these requirements, we recommend you install the Python 3 version of the Anaconda Python distribution.
I strongly recommend Anaconda3 over Anaconda2 due to the fact that SHARPpy is Python 3 compatible now, the fact that Anaconda3 can easily create a Python 2.7 environments if needed, (conda create -n your-name-here python=2) while the reverse can be more troublesome, and of course the looming EOL date for Python 2.7. The latter is particularly true for our community, given the scientific Python stack's essentially universal Python 3 adoption and particularly rapid progress toward going Python 3-only, as many major packages like Numpy (upon which SHARPpy depends), Matplotlib, Pandas, IPython, etc. will be dropping Python 2 support for new releases well before the final EOL, or have already done so.
In any case, I can submit a PR if desired.
This isn't necessarily related to SHARPpy the program, but rather something in the README.md file:
While looking at the main README for the SHARPpy program, I decided to try out the Anaconda Python Distribution, specifically the link provided in the quote below:
However, when I did so, I kept getting a Cloudflare 522 error (Connection timed out). So I tried simply going to the main website URL ( located at https://www.continuum.io/), but it redirected to https://www.anaconda.com/.
The main download page is located at this URL:
Thus, the line that provides the link should be rewritten as such:
Hopefully, this correction would be helpful for those who want to use the SHARPpy program, especially through Anaconda!