-
Notifications
You must be signed in to change notification settings - Fork 167
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
Skip solver at install time #559
Conversation
Does this allow users to update their environments in the future? |
It's like installing a conda-locked environment from scratch. With the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Very clever approach
Do you think this will solve issues with users having an existing |
Yes, it will alleviate solver-derived problems because this solution bypasses it entirely. It might still suffer from other configuration keys, but I can't think of any examples right now. We do need better condarc isolation for conda-standalone in general, but that's separate issue (#542) we need to solve at the conda level first (my idea is to expose the CONDARC_SEARCH_PATH constant as a context key). |
Any other comments @conda/constructor? I will merge later this week. |
@@ -5,6 +5,8 @@ channels: | |||
- http://repo.anaconda.com/pkgs/main/ | |||
specs: | |||
- python | |||
exclude: # [unix] | |||
- tk # [unix] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is interesting as a use case.
Thanks everybody! |
The generated The problem seems in Files You have the remap info already in Replace # base environment file used with conda install --file
# (list of specs/dists to install)
write_env_txt(info, dst_dir, info["_urls"]) with # base environment file used with conda install --file
# (list of specs/dists to install)
write_env_txt(info, dst_dir, final_urls_md5s) Equal change is required for sub environment: replace write_env_txt(info, env_dst_dir, env_info["_urls"]) with write_env_txt(info, env_dst_dir, env_urls_md5) ==> generated installer than works in my cases |
Is it possible to ping #656? |
Description
Comes from and closes #541 (read the issue for more details)
I checked using just
--no-deps
as an install flag, and while the docs hint that it skips the dependency checks... what it actually does is to perform a full solve, and then trim the dependencies from the graph.So instead I am opting for a feature already present in
conda
:@EXPLICIT
installs using the remote URLs. Since the repodata and channel cache is mocked already to make the previous method work, we get almost everything for free.Basically, constructor invokes
conda install --offline --file <file>
, with this file:What we are doing in this PR is changing the file syntax to:
This still goes through the transaction API, and the records are sorted in topological order, so we should be good.
Checklist - did you ...
news
directory (using the template) for the next release's release notes?