-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Adding ODL (Operator Discretization Library). #27504
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/odl/meta.yaml:
For recipes/odl/meta.yaml:
|
I confirm that I agree to be listed as maintainer |
1 similar comment
I confirm that I agree to be listed as maintainer |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/odl/meta.yaml:
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/odl/meta.yaml:
|
I confirm that I agree to be listed as maintainer. |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge/help-python, ready for review! |
@conda-forge-admin, relint |
Hi! This is the friendly automated conda-forge-linting service. I Failed to even lint the recipe, probably because of a conda-smithy bug 😢. This likely indicates a problem in your |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge/help-python, is there any problem that prevents this from being merged? It has been a while since I filed the pull request. I already had some strange error reports, apparently because the recipe-branch had gone out of sync with the mainline. But after updating that, all checks pass again so I really don't see why the recipe couldn't be made into a feedstock. |
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.
Please indent with two spaces like all other recipes.
…mited. Versions older than 3.7 do not seem to be available from Conda anymore anyway (which is why I have not tested them), so this should not really make a difference.
This might be useful as information independent of Github.
9e54c0e
to
6cf601f
Compare
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( For recipes/odl/meta.yaml:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Apparently CondaForge uses only two spaces for each Yaml level.
I'm still not sure if I understand what this does exactly and why ODL specified it, since a pure Python package should generally be host-invariant.
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/odl/meta.yaml:
|
These were removed in previous rounds of linter negotiations, but they are actually needed.
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
This is apparently not needed anymore.
@xhochy thanks. I have adopted all your suggestions, fixed problems in CI, and all the tests pass. |
The build succeeds without that, but the `script` line does utilize pip so I suppose it is technically needed. Co-authored-by: Uwe L. Korn <xhochy@users.noreply.github.com>
@xhochy great, thanks for the help! |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).