-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Add PartSeg recipe #17861
Add PartSeg recipe #17861
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
We are in the process of creating conda packages for napari plugins and related dependencies. Once this recipe passes all the checks, every time a new release goes into PyPI, a PR will be automatically open, with usually very minor need for changes so the process of maintaining this moving forward is small. Also the @conda-forge/napari team will be available to help as needed. You can read a bit more about conda-forge at https://conda-forge.org/#contribute @Czaki, do you agree with being added as a maintainer? |
yes |
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 ( |
Depends on: |
This comment was marked as outdated.
This comment was marked as outdated.
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 ( |
Merged! |
@goanpeca As napari just drop python 3.7 support, then PartSeg will drop support for python 3.7 in a short time. So if support of python 3.7 requires an additional job then this python version could be omitted. |
recipes/PartSeg/meta.yaml
Outdated
skip: true # [py<38] | ||
entry_points: | ||
- PartSeg = PartSeg.launcher_main:main | ||
- Tester = PartSeg.test_widget_main:main |
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.
- Tester = PartSeg.test_widget_main:main | |
- PartSeg-Tester = PartSeg.test_widget_main:main |
@Czaki - is it ok if we rename this script? Tester
is a bit too generic and I am concerned it might clash other tools in either conda-forge or the broader ecosystem.
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.
Delete it. I will also do this in PartSeg
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.
Co-authored-by: Grzegorz Bokota <[email protected]>
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).