-
-
Notifications
You must be signed in to change notification settings - Fork 404
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
reading DataFrame index as kdim or vdim in DataSet #2000
Comments
If I understand you correctly, that should be working on the github master version of holoviews, as of #1946, and will be in the next release. |
Oops; didn't read your question closely enough; I thought you were focusing on using the column name rather than a list of column names, which is what the above PR does. Here, you're saying that we should treat the index as if it were a column. I really find it mystifying why Pandas treats the index so very differently from other columns, but yes, I agree, we should be able to pass the name of the index just like we can pass the name of regular columns. We would presumably have to do that as a special case, unfortunately. Meanwhile you can work around it using the Pandas Your title suggests using the index as a vdim, not just a kdim, which seems unlikely to be useful because an index is clearly a key rather than a value dimension, but we'd probably have to explicitly disallow usage of the index as a vdim if we support it for kdim, so maybe the title is correct as written. |
I agree... it would be very weird using the index as a vdim, but "weird" and "impossible" are different... and it seems weird to build code to explicitly disallow something just because it is weird.
I'm think that this |
Implemented now, see #2164. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
If you try and pass in the name of an index as a kdim for a DataSet, then the DataSet should accept that and allow you to use an index as a kdim.
The text was updated successfully, but these errors were encountered: