kz930 opened a new pull request, #8504:
URL: https://github.com/apache/texera/pull/8504

   ### What changes were proposed in this PR?
   
   Ten charts implement `StandaloneCodeGenerator`: Filled Area, Contour, 
Dumbbell, Time Series, Quiver, Choropleth Map, 3D Scatter, Volcano, Gantt and 
Carpet. What they share is the question they answer, which is how a value 
varies across a domain rather than where a row sits on an axis.
   
   Each emits the plotly its executor emits and writes the page to the file the 
translator names for it, so a plan holding two charts writes two files rather 
than one overwriting the other. A chart that cannot be drawn writes the reason 
to that page rather than printing it: the engine's operator yields a page 
either way, so an export that printed instead would leave the run with no file 
where the workflow had one.
   
   Split out of #8346 on review, which had grown past what one reading can 
hold. That change keeps the charts drawn on a coordinate system.
   
   ### Any related issues, documentation, discussions?
   
   Part of #8325; that issue lists the set in order.
   
   Closes #8503, the task this change is the whole of.
   
   ### How was this PR tested?
   
   Each chart asserts the block it emits in its own spec. Contour, Filled Area 
and Carpet additionally run the generated Python, since what they are worth 
checking is that the page comes out at all on the inputs that cannot be drawn: 
collinear points, a group with no line to draw, a table emptied by dropping 
nulls.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Claude Opus 5)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
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]

Reply via email to