Skip to content

fix(scatter): apply visualMap on large datasets and add house-price example#172

Merged
fahimfaisaal merged 2 commits into
mainfrom
fix/scatter-visualmap-large-datasets
Jun 28, 2026
Merged

fix(scatter): apply visualMap on large datasets and add house-price example#172
fahimfaisaal merged 2 commits into
mainfrom
fix/scatter-visualmap-large-datasets

Conversation

@fahimfaisaal

Copy link
Copy Markdown
Member

Summary

Fixes 2D scatter --visualmap not applying gradient colors on large datasets (e.g. 16k+ points) and adds a house-price CSV example to CI.

Problem

ECharts skips visualMap when scatter series runs in large mode. Vizb always enabled large: true above 2,000 points, so --visualmap showed the gradient control but points stayed a single color.

Changes

  • scatterSeriesLargeOpts() — when visualMap is on, set large: false; otherwise keep the performance path (large: true, threshold 2000)
  • ChartScatter.vuereplaceMerge: ['series', 'visualMap'] so toggling visualMap in the UI clears stale large: true
  • Value-mode scatter tooltip — cross axisPointer when both axes are value type
  • house-price-area2.csv — 16,174-row ECharts house-price demo data
  • CI — deploy matrix entry 07-house-price-area2 with scatter:visualmap

Testing

  • All 378 UI tests pass
  • Pre-commit hooks (format, go-test, ui-gen-check) pass

…xample

- Disable ECharts scatter `large` mode when visualMap is enabled so
  gradient coloring works regardless of point count
- Replace-merge series on ChartScatter updates so UI toggle takes effect
- Use cross axisPointer for value-mode scatter tooltips
- Add house-price-area2.csv example and CI deploy matrix entry
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@fahimfaisaal
fahimfaisaal merged commit 9a8c9b7 into main Jun 28, 2026
13 checks passed
@fahimfaisaal
fahimfaisaal deleted the fix/scatter-visualmap-large-datasets branch June 28, 2026 10:19
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.

2 participants