Fix conda environment yaml parsing errors#4078
Conversation
Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
| primary_language=cls.default_primary_language, | ||
| dependencies=dependencies, | ||
| extra_data=extra_data, | ||
| is_private=True, |
There was a problem hiding this comment.
Conda environment YAMLs are ostly used to export a particular conda environment (this is like a lockfile used in projects) and to import the environment from them to replicate the conda environment. See https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#creating-an-environment-from-an-environment-yml-file and https://docs.conda.io/projects/conda/en/stable/user-guide/concepts/environments.html
This is not a manifest that you can publish a conda package, for that you need build recipes in meta.yaml files, since this manifest is not used to publish packages at all, and only publish environments, I've marked the packages created as private.
pombredanne
left a comment
pombredanne
left a comment
There was a problem hiding this comment.
Thanks! Just a few nits, wrt. repo URLs and private vs. public.
| @@ -0,0 +1,3 @@ | |||
| name: test | |||
| dependencies: | |||
| - https://104.17.92.24:443::flask=1.0.2 | |||
There was a problem hiding this comment.
Is the URL a namespace, or is is rather a repository_url qualifier?
There was a problem hiding this comment.
It is a repository URL qualifier allright, there are common ones like conda-forge, anaconda, bioconda and the URL is usually not specified fully, but denoted by just a reference word.
There was a problem hiding this comment.
Actually I am not sure, this probably requires a bit more research, there are examples which seem more like a namespace, less like a repository URL:
- https://anaconda.org/r/r-terra
conda install r::r-terra - https://anaconda.org/lightsource2-dev/matplotlib
conda install lightsource2-dev::matplotlib
Opening a follow up issue instead.
2 participants
Tasks
Run tests locally to check for errors.