The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-7343-c7e7362a4022d41238227f47ec474dadd225150e has failed. Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run: 10c46f1e4d31893e5f9d12e7c19ff98e79932d04 / Kary Zheng <[email protected]> feat(contour-plot): plot at the default grid size when the field is left alone (#7343) ### What changes were proposed in this PR? Contour Plot's Grid Size is declared an optional string, but the generated code consumes it as `int(<value>)` with no guard, so the only content the operator accepts is an integer literal. Typing `2.5` into a field described as "Grid resolution of the final image" aborts the run with `ValueError: invalid literal for int() with base 10: '2.5'`, and the message names no field, so nothing points back to Grid Size. Any non-integer text does the same. The field is now an `Option[Int]` falling back to the documented default of 10, and the number is emitted directly rather than wrapped in `int()`. The form renders it as a numeric input, so the free text that reached `int()` no longer exists. `contentAs` names the boxed class because `Option` erases its element type; without it a blank would read as 0 rather than as absent. ### Any related issues, documentation, discussions? Closes #7212. Split out of #7233, which covers the same numeric-settings gap in Bullet Chart and Gauge Chart. ### How was this PR tested? `ContourPlotOpDescSpec` covers it: the generated code carries the grid size as a number, falling back to 10 when the field is unset, and emits an explicit value as itself. It also pins the deserialization — a JSON number, the numeric string a workflow saved before the field was numeric, and an absent value read as unset rather than as zero. ``` sbt "WorkflowOperator/testOnly org.apache.texera.amber.operator.visualization.contourPlot.ContourPlotOpDescSpec" ``` Nine cases, all passing. In the UI on main, a Contour Plot fed three numeric columns with Grid Size set to `2.5` aborts the run: <img width="1290" height="918" alt="Screenshot 2026-08-07 at 3 16 14 PM" src="https://github.com/user-attachments/assets/f93c8403-6eac-427c-b16f-9b7af858e786"; /> With this PR the field only accepts a number and the generated code no longer calls `int()` at all. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 5) Co-authored-by: Claude Opus 5 (1M context) <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/31670874286 With regards, GitHub Actions via GitBox
