-
-
Notifications
You must be signed in to change notification settings - Fork 145
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
Gensim - add missing fuzzytm run dependency #442
Conversation
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 @PrimozGodec, can you share a link somewhere explaining the underlying issue for reference? Also, pinging @conda-forge/gensim for advice / awareness / approval. |
Hi @jaimergp, I added the link to PR where I have problems with this missing package in the description conda-forge/orange3-text-feedstock#63. |
For reference, this is the error we can see in that log:
But that's a bit weird, since Looking at the PyPI tarball, in
, so no fuzzytm there either. It looks like a false positive or error in |
Ah wait, 4.3.0 did require it. |
In that case, I'm ok. I'll just give @conda-forge/gensim a couple days so they can see this. Thanks for reporting it! |
@jaimergp, can it be merged? I think @conda-forge/gensim forgot to include this requirement since it was added by mistake to the package (and removed in a later release). Anyway, for 4.3.0 to be used, it must be in the requirements since it is in the PyPI release requirements. This PR prevents me to proceed with conda-forge/orange3-text-feedstock#63 |
Repodata patching should not add dependencies AFAIK. It should only change version constraints or add to run constrains. |
If it was added by mistake (the PR that added the requirement does not import it at all), then I am inclined to leave it as is, and for your use case, add |
@jaimergp, thank you for the suggestion. I didn't think about this solution before. I agree it is the best idea. I am closing this PR. |
Checklist
python show_diff.py
and posted the output as part of the PR.How I found the error. When the bot proposed the new release for the orange3-text package, it didn't go through because of this missing package. Take a look at the building log: conda-forge/orange3-text-feedstock#63