-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add other Recharts chart formats #96
Comments
Hi, and thanks for your interest! Recharts was initially added to ProofWidgets as a proof of concept. Now that ProofWidgets is more stable, and is included in mathlib, we have to carefully consider which JS libraries it bundles (see #19). Having used the Recharts integration recently, I am not very impressed by this particular library. The documentation is not the best, and some common tasks are weirdly difficult. For example, to plot multiple functions in one component, it seems one has to compute the union of their graphs manually (see e.g. here). I would be happy to be proven wrong, but if that's the case we might want to consider switching to another plotting library rather than developing the Recharts API more. One popular option (which we already bundle for GraphDisplay) seems to be d3, with Observable Plot on top. |
I wasn't familiar with Recharts or Observable Plot before now. I started from I've done a lot of work in the past with Vega-lite. You've probably considered it, but it might be another option. Popular, does a lot, and the docs are OK. The problem with the library, and the docs, is that there are so many options and layers, that it's a lot to work through if you don't use it often. I used it with Clojure because it was what Clojure data science people were building on. |
Observable Plots looks good. Maybe this issue should be closed, and I can open another one for plotting with Observable Plots. |
If at some point plotting from Observable Plots is added, should |
It appears that at present ProofWidgets4 only provides the
LineChart
chart format. Other charts--at leastScatterPlot
--would be useful.(I might try adding this myself, if that might be OK, but I thought I'd create an issue for it first.)
The text was updated successfully, but these errors were encountered: