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

   ### What changes were proposed in this PR?
   
   The standalone export of the advanced sklearn trainers now runs the same 
distinct-parameter check as the engine path. Two rows naming one parameter used 
to emit the keyword twice, so the export reported success and the script then 
failed with a SyntaxError. The export now fails and names the parameter.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8757.
   
   ### How was this PR tested?
   
   A new case in `SklearnAdvancedBaseDescSpec` exports an operator with two 
rows naming one parameter and expects the refusal. It fails without the change.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Claude Opus 5.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