Skip to content

Feature/revise readme#27

Closed
VarenyaJ wants to merge 2 commits intomainfrom
feature/revise-readme
Closed

Feature/revise readme#27
VarenyaJ wants to merge 2 commits intomainfrom
feature/revise-readme

Conversation

@VarenyaJ
Copy link
Owner

@VarenyaJ VarenyaJ commented Jul 9, 2025

Add warnings about Conda connection problems due to organization proxies blocking anaconda

@VarenyaJ VarenyaJ requested review from SmartMonkey-git and removed request for SmartMonkey-git July 9, 2025 09:28
Comment on lines +139 to +142
conda activate p5
pip install -r requirements.txt \
-r requirements_scripts.txt \
-r requirements_test.txt
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is wrong, no? There is a file for this.

Copy link
Owner Author

Choose a reason for hiding this comment

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

This only creates the env without the environment.yml. I could have used it but when the proxy error occured I wanted to test if I could first make the environment, and then populate it with packages afterwards (pip is not blocked by Charité)

Copy link
Collaborator

Choose a reason for hiding this comment

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

And why would you want to create the env without the environment.yml?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Just to test if only the environment needs to be created - I recognized that it was a problem with conda being blocked and wanted to isolate the problem of creating the environment without worrying about all the packages we needed to install

export SSL_CERT_FILE=$(python -c "import certifi; print(certifi.where())")

# now retry
conda env create -f requirements/environment.yml -n p5
Copy link
Collaborator

Choose a reason for hiding this comment

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

are you sure -n p5 is needed?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Not necessarily if we're using the environment.yml, this was just carried over when I was writing the amended README after engaging in the later steps

Copy link
Collaborator

@SmartMonkey-git SmartMonkey-git left a comment

Choose a reason for hiding this comment

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

I would not add this to our code base. Its not really a part of the project and bloats our readme.
I dont think we should account for every situation the project might end up in.
You can document this rather on teams.

@VarenyaJ
Copy link
Owner Author

VarenyaJ commented Jul 9, 2025

I would not add this to our code base. Its not really a part of the project and bloats our readme. I dont think we should account for every situation the project might end up in. You can document this rather on teams.

Would you be fine with me adding it to an Issue? I want to keep this documented somewhere, in case someone or one of us runs into this problem again and does not know why it's happening. I feel that it's also good to note since the last group discussion revealed people are weary about conda, and this is just to show that conda is not the problem but some workaround is needed if the network is blocking Anaconda repositories

@SmartMonkey-git
Copy link
Collaborator

Would you be fine with me adding it to an Issue? I want to keep this documented somewhere, in case someone or one of us runs into this problem again and does not know why it's happening. I feel that it's also good to note since the last group discussion revealed people are weary about conda, and this is just to show that conda is not the problem but some workaround is needed if the network is blocking Anaconda repositories

Yes, keep it in teams. Then others of our group can also use it.

@VarenyaJ
Copy link
Owner Author

VarenyaJ commented Jul 9, 2025

Would you be fine with me adding it to an Issue? I want to keep this documented somewhere, in case someone or one of us runs into this problem again and does not know why it's happening. I feel that it's also good to note since the last group discussion revealed people are weary about conda, and this is just to show that conda is not the problem but some workaround is needed if the network is blocking Anaconda repositories

Yes, keep it in teams. Then others of our group can also use it.

I meant that I realize this is bloating the README, so I want to move it to https://github.com/VarenyaJ/P5/issues/10 for later reference. Of course I can also post about it on the Coding channel.

@VarenyaJ VarenyaJ closed this Jul 24, 2025
@VarenyaJ
Copy link
Owner Author

Will revisit later

@VarenyaJ VarenyaJ deleted the feature/revise-readme branch August 18, 2025 11:40
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