feat: add a new ternary contour plot operator by ELin2025 · Pull Request #4193 · apache/texera

ELin2025 · 2026-02-06T23:30:14Z

What changes were proposed in this PR?

This change relates to the addition of a ternary contour plot operator, which visualizes how a scalar value varies as a function of three normalized components that sum to a constant (typically 1 or 100%).

In a ternary contour plot:

Each vertex of the triangular plot represents 100% of one component.
Any interior point represents a mixture of the three components.
Contour lines or color gradients indicate equal values of the measured quantity across different mixtures.

This visualization is useful for identifying regions where the output is optimized or insensitive to changes in the component proportions, as well as for understanding trade-offs between the three variables.

The operator takes in 4 inputs. The first three variables are the components, and the fourth variable is the output that corresponds to the proportion of the the three components.

Any related issues, documentation, discussions?

Needs python library scikit-image
Can be installed using: pip install scikit-image

How was this PR tested?

Tested with existing test cases

Was this PR authored or co-authored using generative AI tooling?

No

Ma77Ball · 2026-02-12T23:01:23Z

@ELin2025 For this PR, I’m seeing several changes that don’t seem related to the ternary contour plot work. Could we either remove those from this PR or move them into a separate branch/PR so they can be reviewed and explained more clearly?

ELin2025 · 2026-02-12T23:58:28Z

@Ma77Ball Just fixed the commits, should just be my files now

chenlica · 2026-02-13T04:53:21Z

@Ma77Ball After you review and approve it, you can let @aicam do the review, approval, and merge.

Ma77Ball

Overall looks good. I left a few comments for discussion and resolution.

Ma77Ball · 2026-02-16T11:59:25Z

...ala/org/apache/texera/amber/operator/visualization/ternaryContour/TernaryContourOpDesc.scala

+    val outputSchema = Schema()
+      .add("html-content", AttributeType.STRING)
+    Map(operatorInfo.outputPorts.head.id -> outputSchema)
+    Map(operatorInfo.outputPorts.head.id -> outputSchema)


Remove duplicate line

@ELin2025 removed duplicate line: "Map(operatorInfo.outputPorts.head.id -> outputSchema)"

Ma77Ball · 2026-02-16T12:05:15Z

...ala/org/apache/texera/amber/operator/visualization/ternaryContour/TernaryContourOpDesc.scala

+      pyb"""
+         |from pytexera import *
+         |
+         |import plotly.express as px


This line can be removed as it is not used.

@ELin2025 removed "|import plotly.express as px" as it is not used in the operator

Ma77Ball · 2026-02-16T12:10:45Z

...ala/org/apache/texera/amber/operator/visualization/ternaryContour/TernaryContourOpDesc.scala

+  def createPlotlyFigure(): PythonTemplateBuilder = {
+    pyb"""
+         |        A = table[$firstVariable].to_numpy()
+         |        B = table[$secondVariable].to_numpy()
+         |        C = table[$thirdVariable].to_numpy()
+         |        Z = table[$fourthVariable].to_numpy()
+         |        fig = ff.create_ternary_contour(np.array([A,B,C]), Z, pole_labels=[$firstVariable, $secondVariable, $thirdVariable], interp_mode='cartesian')


The plotly ternary_contour requires scikit-image. This should be added to the operator-requirements.txt.

@ELin2025 added in requirements.txt

Ma77Ball · 2026-02-16T12:13:04Z

...ala/org/apache/texera/amber/operator/visualization/ternaryContour/TernaryContourOpDesc.scala

+      operatorDescription =
+        "A ternary contour plot shows how a measured value changes across all mixtures of three components that always sum to a constant (usually 100%).",


There is also an info button that can be added to operators. I recommend using it for long descriptions.

@ELin2025 made the description shorter.

ELin2025 · 2026-02-17T00:53:20Z

@Ma77Ball I have made the changes

Ma77Ball · 2026-02-19T01:31:41Z

In the future, please respond to the comments and mention a little of what you did to address the issue after you can hit resolve comment at the bottom of each. This action helps with the review process.

Ma77Ball · 2026-02-19T01:42:57Z

LGTM! @aicam, please review and approve the workflows and merge.

chenlica · 2026-02-19T04:56:53Z

@aicam Please review it.

ELin2025 · 2026-02-23T02:40:06Z

@Ma77Ball @aicam I've downgraded the scikit-image library to be compatible with Python 3.10. All build tests have passed on my local branch.

github-actions bot added frontend Changes related to the frontend GUI common engine dependencies Pull requests that update a dependency file python ci changes related to CI labels Feb 6, 2026

ELin2025 force-pushed the ternary-contour branch from 1894729 to 6d5252b Compare February 12, 2026 23:52

github-actions bot removed engine dependencies Pull requests that update a dependency file python ci changes related to CI labels Feb 12, 2026

ELin2025 force-pushed the ternary-contour branch from 6d5252b to 1767b49 Compare February 12, 2026 23:57

chenlica requested a review from aicam February 13, 2026 04:52

Ma77Ball approved these changes Feb 16, 2026

View reviewed changes

ELin2025 force-pushed the ternary-contour branch from 0e551bd to 20aae0b Compare February 17, 2026 00:51

github-actions bot added the engine label Feb 17, 2026

ELin2025 force-pushed the ternary-contour branch from 20aae0b to ac7fd23 Compare February 19, 2026 00:41

ELin2025 force-pushed the ternary-contour branch from ac7fd23 to fa4be85 Compare February 23, 2026 02:01

ELin2025 force-pushed the ternary-contour branch from de6e995 to c15434d Compare February 23, 2026 19:12

ELin2025 added 3 commits February 23, 2026 11:45

added ternary contour op

2959624

scala format fix

0717639

reconfigured ternary contour op to most recent PR merge

44d0c9c

ELin2025 added 2 commits February 23, 2026 11:45

made changes from PR comments

01f5491

downgraded scikit-image library to be compatible with Python 3.10

689d988

ELin2025 force-pushed the ternary-contour branch from c15434d to 689d988 Compare February 23, 2026 19:45

		operatorDescription =
		"A ternary contour plot shows how a measured value changes across all mixtures of three components that always sum to a constant (usually 100%).",

Comments

Conversation

ELin2025 commented Feb 6, 2026

What changes were proposed in this PR?

Any related issues, documentation, discussions?

How was this PR tested?

Was this PR authored or co-authored using generative AI tooling?

Uh oh!

Ma77Ball commented Feb 12, 2026

Uh oh!

ELin2025 commented Feb 12, 2026

Uh oh!

chenlica commented Feb 13, 2026

Uh oh!

Ma77Ball left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ELin2025 commented Feb 17, 2026

Uh oh!

Ma77Ball commented Feb 19, 2026

Uh oh!

Ma77Ball commented Feb 19, 2026

Uh oh!

chenlica commented Feb 19, 2026

Uh oh!

ELin2025 commented Feb 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants