-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: move Trellis plotting under Plotting #8351
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
Comments
I noticed that the other day. I was wondering if I messed it up when I did the reorganization. I'll move it under plotting. |
I know the PR is already merged, but some comments:
|
All good points. It does really feel buried at the end of that massive page. Thoughts on putting a warning about it possibly being deprecated in the future. I can also try to do all the examples in seaborn to show a side by side and help people transition away. I don't use ggplot, but it would presumably be capable. |
If we deprecate it, that would be a great way to document it (current example with pandas trellis interface -> how to do it in seaborn/ggplot) But for the actual discussion on deprecating it, I will reinitiate it in #3445 |
@jorisvandenbossche @TomAugspurger this all sounds fine as an aside, I had a discussion with cc @tacaswell These guys are in the process of proposing essentially an API that would allow something like this:
IOW, they would in theory handle all of the messy details that pandas now handles for example resample time-series and such (via callbacks) for example frequency resetting when zooming. So pandas would just be the interface and the implementation would move to these engines. So I think they will open a discussion soon on this. |
@jreback That sounds interesting! Although I don't think it is really related to this issue, as you speak of |
@jorisvandenbossche correct....just wanted to mention (and don't need a new issue at this point) |
As this is not really a fleshed out section, kind of odd to have it as a top-level
@TomAugspurger
@jorisvandenbossche
The text was updated successfully, but these errors were encountered: