facebookresearch/beanmachine

update plotting components

ndmlny-qs opened this issue · 0 comments

Issue Description

We currently have a fragmented implementation in the documentation where pages that are not tutorials directly use components in the Plotting.jsx file in their mdx file. For an example, see https://github.com/facebookresearch/beanmachine/blob/main/docs/overview/analysis/analysis.mdx where we import the BokehFigure component and use it directly in the analysis page compared to https://github.com/facebookresearch/beanmachine/blob/gh-pages/_src/overview/tutorials/Coin_flipping/CoinFlipping.mdx where we are creating plots in a different .jsx file that is imported into the mdx file.

The goal of this change is to update the way in which we plot things using Bokeh or Plotly in the MDX conversion. This change also has the side effect that it will get us closer to removing BokehJS from downloading on each page of the site, and only use it when needed to plot items, see #1348 for more discussion.

Additional Context

See the following for more discussion.