spatialdata v0.8.0#36
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 ( |
| {% set version = "0.7.3" %} | ||
| {% set python_min = "3.11" %} | ||
| {% set version = "0.8.0" %} | ||
| {% set python_min = "3.12" %} |
| - annsel >=0.1.2 | ||
| - click | ||
| - dask >=2025.12.0,<2026.1.2 | ||
| - dask >=2026.3.0 |
There was a problem hiding this comment.
Upstream uses dask in pyproject.toml but grayskull prefers dask-core. This caused the initial conda solver error (since grayskull applied the updated pin to dask-core but left the existing incompatible pin on dask).
Could/should this recipe use dask-core over dask? I'm not familiar enough with dask to answer that, so for consistency I continued to require dask. spatialdata maintainers, please feel free to assess and update as desired.
https://github.com/conda-forge/dask-core-feedstock
https://github.com/conda-forge/dask-feedstock
There was a problem hiding this comment.
Thanks for flagging this. For dask vs dask-core see here dask/distributed#962 (comment). AFAIU dask should be including both dask-core (equivalent of PyPI's dask) and distributed. Since we depend explicitly on distributed (btw we will drop the dependency after we merge and release scverse/spatialdata#1106), it seems like we should be using dask-core here.
It is very likely that the current package version for this feedstock is out of date.
Checklist before merging this PR:
license_fileis packagedInformation about this PR:
@conda-forge-admin,please add bot automergein the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.bot-rerunlabel to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase@conda-forge-admin, please rerun botin a PR comment to have theconda-forge-adminadd it for you.Pending Dependency Version Updates
Here is a list of all the pending dependency version updates for this repo. Please double check all dependencies before merging.
Dependency Analysis
Please note that this analysis is highly experimental. The aim here is to make maintenance easier by inspecting the package's dependencies. Importantly this analysis does not support optional dependencies, please double check those before making changes. If you do not want hinting of this kind ever please add
bot: inspection: disabledto yourconda-forge.yml. If you encounter issues with this feature please ping the bot teamconda-forge/bot.Analysis by grayskull shows a discrepancy between it and the the package's stated requirements in the meta.yaml.
Packages found by grayskull but not in the meta.yaml:
Packages found in the meta.yaml but not found by grayskull:
This PR was created by the conda-forge-bot. The conda-forge-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/conda-forge/conda-forge-bot/actions/runs/28630717174 - please use this URL for debugging.