rusackas opened a new pull request, #42300:
URL: https://github.com/apache/superset/pull/42300

   ### SUMMARY
   When Y axis truncation is on and a value falls outside the configured 
min/max, ECharts was dropping that point (and the line segments around it) 
instead of clipping it at the boundary. This clamps series values to the 
configured Y axis bounds before handing them to ECharts, so a spike gets a flat 
top instead of a gap. Covers Line, Area, Smooth, Step and Bar series in the 
shared Timeseries chart.
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   See the screenshots on #27449, this reproduces the same "expected" flat-top 
behavior described there.
   
   ### TESTING INSTRUCTIONS
   Added a test that pins the bug: a line series with a value of 1000 and Y 
axis bounds `[0, 10]` used to drop that point; now the point stays in the 
series data, clamped to 10. Ran via `npx jest 
plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts`.
   
   Manually: create a line/area/bar chart with a big outlier, turn on "Truncate 
Y Axis" with a max below the outlier, confirm the line still connects through a 
flat top rather than a gap.
   
   ### ADDITIONAL INFORMATION
   - [x] Has associated issue: Fixes #27449
   - [ ] Required feature flags:
   - [x] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to