Skip to content

docs: expand the Plotly page with 14 chart-type recipes#6777

Open
amsraman wants to merge 3 commits into
mainfrom
docs/plotly-chart-recipes
Open

docs: expand the Plotly page with 14 chart-type recipes#6777
amsraman wants to merge 3 commits into
mainfrom
docs/plotly-chart-recipes

Conversation

@amsraman

Copy link
Copy Markdown
Contributor

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:

  • Plotly Express section gains: Bubble Chart, Gantt Chart (px.timeline), Sunburst Chart, Funnel Chart
  • New Financial Charts group: Candlestick, Waterfall, Bullet (go.Indicator)
  • New Statistical Charts group: Continuous Error Bands (reversed-bound fill="toself" technique)
  • New Maps group: Geo Map (go.Scattergeo + update_geos), Scatter Map (px.scatter_map)
  • New Tables and Diagrams group: go.Table, Sankey Diagram
  • New 3D Charts group: 3D Scatter (px.scatter_3d), 3D Axis configuration (go.Mesh3d + scene axes)

Quality notes

  • Every new example is a self-contained python demo exec block 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).
  • Source snippets that fetched remote CSVs/JSON at import time, required geopandas, or used deprecated APIs (ff.create_gantt) were deliberately dropped or replaced with their modern equivalents.
  • One source snippet mislabeled its data ("The Great Recession" over 2016 AAPL data) — retitled.

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

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>
@amsraman
amsraman requested review from a team and Alek99 as code owners July 15, 2026 22:40
@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This 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 funnel_chart naming inconsistency have both been addressed in this revision.

  • 14 new recipes added covering Bubble, Gantt, Sunburst, Funnel, Candlestick, Waterfall, Bullet, Continuous Error Bands, Geo Map, Scatter Map, Table, Sankey, 3D Scatter, and 3D Axis charts — each as a self-contained python demo exec block with no network dependencies.
  • Minor naming inconsistency remains within the new sections: plotly_table and plotly_sankey_diagram carry the plotly_ prefix while the other new section functions (candlestick_chart, waterfall_chart, etc.) do not.

Confidence Score: 5/5

Safe 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

Filename Overview
docs/library/graphing/other-charts/plotly.md Adds 14 new self-contained chart-type recipes across six new sections; previous review issues (remote CSV fetch, funnel naming) are resolved in this revision.

Reviews (3): Last reviewed commit: "Apply ruff-format to inline candlestick ..." | Re-trigger Greptile

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment on lines +272 to +273
candles = pd.read_csv(
"https://raw.githubusercontent.com/plotly/datasets/master/finance-charts-apple.csv"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

Comment thread docs/library/graphing/other-charts/plotly.md Outdated
Comment thread docs/library/graphing/other-charts/plotly.md Outdated
@codspeed-hq

codspeed-hq Bot commented Jul 15, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 26 untouched benchmarks
⏩ 8 skipped benchmarks1


Comparing docs/plotly-chart-recipes (bb5c1b7) with main (7e80931)

Open in CodSpeed

Footnotes

  1. 8 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant