docs: expand the Plotly page with 14 chart-type recipes#6777
Conversation
Converts the Plotly chart recipes from Reflex Build's AI builder knowledge base into doc sections: bubble, gantt, sunburst, funnel join the Plotly Express section; new Financial (candlestick, waterfall, bullet), Statistical (continuous error bands), Maps (geo, scatter map), Tables and Diagrams (go.Table, sankey), and 3D (scatter, axis config) groups. Every demo block is self-contained and verified to execute. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Greptile SummaryThis PR expands the Plotly docs page with 14 new self-contained chart-type recipes grouped into six new sections (Financial Charts, Statistical Charts, Maps, Tables and Diagrams, 3D Charts, plus four new Plotly Express subsections). All examples use inline data — previous review feedback about a remote CSV fetch in the Candlestick example and the
Confidence Score: 5/5Safe to merge — all changes are documentation additions with no runtime or application logic changes. The change is purely additive documentation: 14 new self-contained code examples with hardcoded inline data. Previously raised issues (remote network fetch at module level, funnel naming) are fixed. No application code is touched. No files require special attention; the single changed file is documentation only. Important Files Changed
Reviews (3): Last reviewed commit: "Apply ruff-format to inline candlestick ..." | Re-trigger Greptile |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6fdf0b0543
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| candles = pd.read_csv( | ||
| "https://raw.githubusercontent.com/plotly/datasets/master/finance-charts-apple.csv" |
There was a problem hiding this comment.
Avoid fetching remote data during docs render
This python demo exec block is executed while the docs page is rendered (_render_demo runs _exec_code for these fences in docs/app/reflex_docs/docgen_pipeline.py), so reading the CSV from GitHub here makes the Plotly docs page depend on external network availability and latency. In offline/local builds or transient GitHub failures this single example prevents the page from rendering; use a small inline DataFrame or a checked-in docs data file instead.
Useful? React with 👍 / 👎.
Merging this PR will not alter performance
Comparing Footnotes
|
What
Second batch of upstreaming Reflex Build's AI-builder knowledge base into the official docs (first: #6775). This one converts the Plotly chart recipes into 14 new sections on
docs/library/graphing/other-charts/plotly.md:px.timeline), Sunburst Chart, Funnel Chartgo.Indicator)fill="toself"technique)go.Scattergeo+update_geos), Scatter Map (px.scatter_map)go.Table, Sankey Diagrampx.scatter_3d), 3D Axis configuration (go.Mesh3d+ scene axes)Quality notes
python demo execblock in the page's existing style (module-level figure +def *_chart()); all blocks on the page were exec-verified (the only non-running one is the pre-existing surface-plot example that reads a docs-site-local CSV).geopandas, or used deprecated APIs (ff.create_gantt) were deliberately dropped or replaced with their modern equivalents.If the maintainers would rather see the new groups split onto separate pages (e.g.
plotly-maps.md), happy to restructure — one page keeps the diff reviewable for now.🤖 Generated with Claude Code