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

Rename virtual environment - use different name to “venv” since it can be confusing for learners #328

Closed
anenadic opened this issue Mar 26, 2024 · 3 comments
Milestone

Comments

@anenadic
Copy link
Collaborator

We could have “venv*” in gitignore and “venv_xyz” as the name of the venv. Despite the convention, we should go with clarity for learners.

@anenadic anenadic added this to the April2024-beta milestone Apr 11, 2024
@bielsnohr
Copy link
Collaborator

Was there specific feedback that venv is a confusing name? I don't think adding something at the end will necessarily make it that much clearer. I could see virtual-env being perhaps slightly more clear; however, as you mention this goes against the established convention, and I think that is a fine line to balance. Further discussion in upcoming meeting that I will add here once we have it.

@anenadic
Copy link
Collaborator Author

anenadic commented May 8, 2024

Yes, there was a specific feedback that it was confusing throughout the last workshop in March 2024. I was thinking of changing the whole material, not just adding a note at the end. Even though it may not be conventional (i.e. the convention being to use names such as "venv" or ".venv") - Greg Wilson always said "never be scared to sacrifice truth for clarity", in this case "convention over clarity".

@anenadic
Copy link
Collaborator Author

anenadic commented May 8, 2024

Discussions at the maintainers call on 2024-05-08:
Renaming the venv directory storing the virtual environment to something more meaningful for users but deviating from the convention. How far do you go with breaking the convertion to make it easier for people? Are we seeing this repeatedly in multiple workshops? Do we want to make changes based on just one workshop? Do we need more evidence to make this kind of change? Put this in instructor notes as a course of confusion? Clarification at the start, making it more clear in the rest of the material?

Conclusion: stay with the convention for the time being but leave it as an open issue. Ask people running workshops to pay attention to this next time they run the course and see if more learners are struggling. Also add a reminder to activate a virtual environment at the start of each section - see #346 ?

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

No branches or pull requests

2 participants