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

   ### What changes were proposed in this PR?
   
   The operator handed its three columns to scipy exactly as they arrived, so 
three ordinary tables ended the run: an empty one, one with a blank in x or y, 
and one whose points all fall on a line. It had no `render_error` and no guard 
of any kind.
   
   It now drops the rows missing any of the three and answers an emptied table 
with an error page, the wording Bar Chart already uses. A blank in z is part of 
the same drop: it interpolates to an all-NaN grid, which draws an empty chart 
with nothing to say why.
   
   The degenerate point set is guarded by a rank test on the unique (x, y) 
pairs rather than by catching the QhullError, so a genuine failure inside 
griddata still surfaces as a failure. Points spanning a plane are exactly the 
ones Qhull can start from: three points not on a line are enough, and two never 
are.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8076
   
   ### How was this PR tested?
   
   Three cases added to ContourPlotOpDescSpec, over the emitted code. I also 
ran the guarded body against each failing table in scipy: the empty table and 
the all-blank column reach the error page, a blank row is dropped and the 
remaining four points plot, four collinear points and two points reach the line 
message, and three non-collinear points and a four-point square plot as before.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Claude Opus 5)


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