-
-
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
Adds pyrtools #25506
Adds pyrtools #25506
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/plenoptic:
For recipes/pyrtools:
|
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/pyrtools:
|
@conda-forge-admin, please ping conda-forge/help-python Hi conda forge! I'm trying to get this ready for review, but I don't understand the lint error I'm getting for |
Hi! This is the friendly automated conda-forge-webservice. I was asked to ping @conda-forge/help-python and so here I am doing that. |
Hi! This is the friendly automated conda-forge-webservice. I was asked to ping @conda-forge/help-python and so here I am doing that. |
And I'm not sure if the version constraint issue is related to or independent of the failing OSX and Windows builds -- building succeeds for me on github actions (e.g., ) when I build wheels for pip. I'm not familiar enough with what conda is doing to understand why it would fail here. I'd appreciate any help figuring this out! |
The Windows build is failing and I think it's because I require pytorch, the windows build for which is hosted on the |
Yes, you can just ignore the failure. |
Okay, that makes sense. What are my next steps then? Is there some way to fix the failing OSX build or should I ignore that as well? And if you don't recommend installing from multiple channels, what do you recommend doing for pytorch? |
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/pyrtools:
For recipes/pyrtools:
|
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/staged-recipes I believe this is ready to be merged: I have fixed all the linters issues and my OSX and Linux builds are succeeding. The Windows build is still failing, because it requires conda-forge/pytorch-cpu-feedstock#32, so will not succeed for now. Let me know if there are any other changes I need to make! |
To help direct your pull request to the best reviewers, please mention a topic-specifc team if your recipe matches any of the following: conda-forge/help-c-cpp, conda-forge/help-cdts, conda-forge/help-go, conda-forge/help-java, conda-forge/help-julia, conda-forge/help-nodejs, conda-forge/help-perl, conda-forge/help-python, conda-forge/help-python-c, conda-forge/help-r, conda-forge/help-ruby,or conda-forge/help-rust. Thanks! |
@conda-forge/help-python is the proper team to notify, I believe. |
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.
Can you please separate these packages? As only one is noarch I'm hesitant to merge with failing Windows CI.
Okay. Since plenoptic (which also relies on pytorch and thus has the failing windows build) depends on pyrtools, is the way to do this to remove plenoptic from this PR, merge pyrtools, and then open a new PR with plenoptic? |
Yes! |
Following feedback from conda-forge maintainers, splitting pyrtools and plenoptic into two (sequential) PRs
Alright, I went ahead and removed plenoptic! |
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 ( |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).plenoptic depends on pyrtools, and I'm the maintainer of both. They are used in visual neuroscience for building and understanding computational models of neurons and/or perception.