- Notifications
You must be signed in to change notification settings - Fork1.2k
Always use environment path when running conda environment commands#24807
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@eleanorjboyd While I work on getting the CLA approved, would really appreciate if someone could give this a look! |
@jpcorreia99 This is a great idea to always use the prefix for the |
Head branch was pushed to by a user without write access
jpcorreia99 commentedFeb 12, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
hey@karthiknadig could you please reapprove the PR 🙏 Had to add an extra commit due to a unit test I had missed. CLA has been signed! (and you may want to remove the other reviewer if possible) Also, is there documentation regarding the release cycle of the extension? Would be interested in knowing when I can expect this change to be available. Thank you for the review! |
@microsoft-github-policy-service agree company="Palantir" |
@karthiknadig little ping here |
Hey@karthiknadig are the other two reviews still necessary? I just tagged the people who I've seen contributing the most recently to the extension |
79e8a13
intomicrosoft:mainUh oh!
There was an error while loading.Please reload this page.
Attempt at fixing#24585
There are many edge scenarious where refering to the name of the environment rather than the path can cause breaks in the extension. Some examples
1 -If we have two anonymous environments with the same name in different folders
/path1/my-env
/path2/my-env (where my active vscode python interpreter is)
by using conda -n my-env it'll always use the first env.
2 -Some times people avoid actually activating their conda envs when using conda-packhttps://github.com/conda/conda-pack
This is because the activation scripts are known to be flaky and not very reliable
3 -The environment may have been created by a conda-compliant replacement
Therefore conda itself is not aware of it by name but can work with it properly using the path. This is the case ofhawk or frankly anyone building their own conda package manager on top ofrattler.
Some of these points are also hinted at#24627 (comment) , and supported by a conda maintainer in#24585 (comment)
This PR has a minimal attempt at changing that by always forcing -p usage