-
-
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
cellprofiler 3.1.9 #9538
cellprofiler 3.1.9 #9538
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/cellprofiler:
|
recipes/cellprofiler/meta.yaml
Outdated
- wxpython | ||
- joblib | ||
- mahotas | ||
- matplotlib==2.2.3 |
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.
I don't think this strict pin is necessary.
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.
yes, it needs to be > 2.0.0, so I will change it to >=2.0
recipes/cellprofiler/meta.yaml
Outdated
- javabridge | ||
- prokaryote | ||
- python-bioformats | ||
- pyzmq=15.3.0 |
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.
relax the pin?
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.
I will try relax it see what happens. it builds find locally for now.
recipes/cellprofiler/meta.yaml
Outdated
- pyzmq=15.3.0 | ||
- raven | ||
- requests | ||
- scikit-image=0.14.0 |
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.
- scikit-image 0.14.*
recipes/cellprofiler/meta.yaml
Outdated
- scipy | ||
# - six==1.11.0 | ||
- pillow | ||
- networkx==2.2 |
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.
- networkx 2.2.*
recipes/cellprofiler/meta.yaml
Outdated
- pillow | ||
- networkx==2.2 | ||
- vigra | ||
- pytest |
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.
Is pytest really a runtime dependency?
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.
i moved it to host:
as I can see setup.py requires it
recipes/cellprofiler/meta.yaml
Outdated
- cellprofiler | ||
|
||
about: | ||
summary: "CellProfiler is free, open-source software" |
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 add license, doc_url etc and under extras you as a maintainer.
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. I have not pushed the actual license file with the correct content yet. I will do that soon
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/cellprofiler:
|
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/cellprofiler:
For recipes/cellprofiler:
|
I have made another commit and it builds fine locally. but I still used the javabridge package from the old 'free' channel. I will have a look at #7701 see what I can do. Yi |
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 ( |
Hi @bgruening , a couple of questions, I tried to install javabridge to local channel using its own conda recipe mentioned in #7710, then build and install cellprofiler. It seems ok, so is this the correct approach for now, since javabridge is not yet in conda-forge? is that possible to write the javabridge installation from source within cellprofiler's meta.yaml? Also, another issue is with the package wxPython, conda-forge only has version 4.0.6, but cellprofiler only works with 3.0.2 for the release3.1.8 thanks very much, |
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/cellprofiler:
|
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
recipes/cellprofiler/meta.yaml
Outdated
version: "{{ version }}" | ||
|
||
source: | ||
git_url: "https://github.com/CellProfiler/CellProfiler.git" |
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.
here you should use a tarball
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/cellprofiler:
|
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 ( |
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/cellprofiler:
|
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 ( |
Hi friend! We really, really, really appreciate that you have taken the time to make a PR on In an effort to maintain this repository and increase the signal-to-noise for open PRs, the maintainers of If you'd like to keep it open, please comment/push and we will be happy to oblige! Note that very old PRs will likely need to be rebased on master so that they can be rebuilt with the most recent CI scripts. If you have any trouble, or we missed reviewing this PR in the first place (sorry!), feel free to ping the team using a special command in a comment on the PR to get the attention of the Cheers and thank you for contributing to this community effort! |
Hi @bgruening ! Sorry, I completely missed your ping and just saw this again 😏 The javabridge issue seemed insurmountable at the time so I ended up using Docker containers directly provided by CellProfiler in the end. Hope you and loved ones are well. |
All good @drpatelh! Conda: https://anaconda.org/bioconda/cellprofiler Thanks to @sunyi000 and @beatrizserrano! |
Checklist
url
) rather than a repo (e.g.git_url
) is used in yourrecipe (see
here
for more details)