-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
param dependency out-of-date? #1019
Comments
Hi @patricktokeeffe ! Thanks for this nice bug report, appreciate the work you put in there 👍 You're right, the minimum version of Param should be 1.12.0. The error message is not quite right though, you are seeing this error as I have seen that you attempted to make a Pull Request. However, you seem to have pointed it to your own fork rather than to this repository. On the image below, it shouldn't be Also your PR is updating the build dependencies, while it should update the runtime dependencies. Let me give you some more context. Param is one of the build dependencies as it has a Lines 30 to 38 in 5129d1f
As you can see Param is not in this list, which is an oversight as it is effectively a direct dependency of hvPlot (it gets anyway installed because it's a dependency of Panel and HoloViews). So to fix this issue you would need to add Let us know if you are still interested in fixing that :) |
Fixed by #1227 |
I experienced problems using hvPlot versions 0.8.0, 0.8.1 and 0.8.2 and the resolution was to manually update the
param
module to version 1.12.0. Seems like dependency version is out-of-date:https://github.com/holoviz/hvplot/blob/main/pyproject.toml:
ALL software version info
Environment
`pip list` (after discovering the issue)
`pip list` (after upgrade to 0.8.1)
Description of expected behavior and the observed behavior
My system is older but I wanted to try out the data explorer. I upgraded to the minimum version specified in the documentation
This resulted in an
AttributeError
. So I upgraded to 0.8.1 instead and received a completely different error.So then I upgraded to the latest version, but I continue to receive a strange error.
However, searching online turned up this holoviz Discourse comment, which had the resolution: update
param
to 1.12.0.I progressively tried each of the major release version of
param
: 1.11.0, 1.11.1, 1.12.0 and determined 1.12.0 is the minimum required.Complete, minimal, self-contained example code that reproduces the issue
version 0.8.0
version 0.8.1
version 0.8.2
version 0.8.2 + param==1.11.0 (next major version)
version 0.8.2 + param==1.11.1
version 0.8.2 + param=1.12.0
Stack traceback and/or browser JavaScript console output
Screenshots or screencasts of the bug in action
The text was updated successfully, but these errors were encountered: