-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Document --platform
flag for conda [env] create
#14181
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
Conversation
However, in some cases you might want to create an environment for a | ||
different target platform or architecture. To do so, use the | ||
``--platform`` flag available in the ``conda create`` and | ||
``conda env create`` commands. The value must be a valid platform. |
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.
Do we have a list of valid platforms in the documentation? Would be a nice thing to link to.
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.
Looks like our CLI help has them listed. https://docs.conda.io/projects/conda/en/latest/commands/env/create.html#named-arguments
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.
Link added.
Thanks for picking up my open issue! 🚀 Couple suggestions for improvement, but it looks good. |
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.
+1 on all of Travis' comments, plus a few of my own!
- The environment can be solved, but extraction and linking fails due | ||
filesystem limitations (case insensitive systems, incompatible paths, | ||
etc). The only workaround here is to use ``--dry-run --json`` to obtain | ||
the solution and process the payload into a lockfile that can be shared |
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.
Do we have docs on lockfiles and how to use them? Might be a useful link here.
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.
Added a new section because I couldn't find one.
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.
Thanks for that!
However, in some cases you might want to create an environment for a | ||
different target platform or architecture. To do so, use the | ||
``--platform`` flag available in the ``conda create`` and | ||
``conda env create`` commands. The value must be a valid platform. |
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.
Looks like our CLI help has them listed. https://docs.conda.io/projects/conda/en/latest/commands/env/create.html#named-arguments
Co-authored-by: Travis Hathaway <travis.j.hathaway@gmail.com> Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
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.
Just a few more comments on my end for your new section, Jaime
They consist of a ``@EXPLICIT`` header plus a list of conda package URLs optionally followed | ||
by their MD5 or SHA256 hash. | ||
|
||
They can be obtained from existing environments via ``conda list --explicit``, as seen in ... |
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.
Don't forget to put something here!
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.
@jaimergp This seems empty still?
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.
Oops, addressed!
- The environment can be solved, but extraction and linking fails due | ||
filesystem limitations (case insensitive systems, incompatible paths, | ||
etc). The only workaround here is to use ``--dry-run --json`` to obtain | ||
the solution and process the payload into a lockfile that can be shared |
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.
Thanks for that!
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
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 needs an update per https://github.com/conda/conda/pull/14181/files#r1737227491
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Description
Closes #14100
Checklist - did you ...
news
directory (using the template) for the next release's release notes?