rapidsai / cuxfilter

GPU accelerated cross filtering with cuDF.
https://docs.rapids.ai/api/cuxfilter/stable/
Apache License 2.0
273 stars 67 forks source link

Refactor to use holoviews powered bar charts #494

Closed AjayThorve closed 1 year ago

AjayThorve commented 1 year ago

This PR refactors the bokeh charts implementation, and uses holoviews to generate the charts, while also using box-selects instead of range_sliders. This was done for 2 reasons, stability of holoviews api, which also is much more efficient, and to use streams + custom callbacks to link all charts with bar_charts.

Some of the other notable changes:

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

AjayThorve commented 1 year ago

Lot of changes! Whats the quick rational for removing the bokeh line chart?

Updated the description,

Adding to it: The current implementation had two line charts, one under bokeh, and another under datashader. Removed one to avoid confusion, and bokeh line's implementation that cuxfilter had wasn't really accomplishing anything that bar charts weren't.

AjayThorve commented 1 year ago

/merge