Skip to content

Conversation

silil
Copy link
Collaborator

@silil silil commented Jun 4, 2019

adding more context on anaconda, miniconda, pyenv and pycharm installation.

@silil silil requested review from nanounanue and rayidghani June 4, 2019 12:50
```
curl https://pyenv.run | bash
```

Copy link
Collaborator

Choose a reason for hiding this comment

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

The files above seem to duplicate some of the information already in

https://github.com/dssg/hitchhikers-guide/tree/master/sources/curriculum/setup/software-setup

Could you please combine the two versions in one place.


Download miniconda from [Conda site, miniconda distribution]( https://docs.conda.io/en/latest/miniconda.html) and follow the instructions.

+ Installing pyenv:
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do you need both anaconda and pyenv?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You don't, its just another option

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah! Ok! So maybe we should write that at the beginning, something that clarifies that.

Copy link
Collaborator

Choose a reason for hiding this comment

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

When combining with the vm section in https://github.com/dssg/hitchhikers-guide/tree/master/sources/curriculum/setup/software-setup this should get solved.


**Why not just a text editor**

Sublime and Atom have become very common for developing scripts, they even let you load a project so that you can access all your scripts on an easy way, but they lack the debug functionality.
Copy link
Contributor

@nanounanue nanounanue Jun 6, 2019

Choose a reason for hiding this comment

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

I don't agree with this. Actually you can argue just the contrary. We should remove this paragraph.

(Quite open for discuss this)

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm also in favour of removing this.

Copy link
Contributor

@nanounanue nanounanue Jun 7, 2019

Choose a reason for hiding this comment

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

Any update on this? @silil

Also, now there are conflicts

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