Skip to content

Extending te_variable functions - #126

Closed
loumalouomega wants to merge 2 commits into
codeplea:masterfrom
loumalouomega:master
Closed

Extending te_variable functions#126
loumalouomega wants to merge 2 commits into
codeplea:masterfrom
loumalouomega:master

Conversation

@loumalouomega

@loumalouomega loumalouomega commented Jun 4, 2025

Copy link
Copy Markdown

Adding:

    {"zIsEqual", IsEqual,               TE_FUNCTION2 | TE_FLAG_PURE, 0},
    {"zIsGreater", IsGreater,           TE_FUNCTION2 | TE_FLAG_PURE, 0},
    {"zIsGreaterEqual", IsGreaterEqual, TE_FUNCTION2 | TE_FLAG_PURE, 0},
    {"zIsLess", IsLess,                 TE_FUNCTION2 | TE_FLAG_PURE, 0},
    {"zIsLessEqual", IsLessEqual,       TE_FUNCTION2 | TE_FLAG_PURE, 0},

This bring the changes we have added to our project https://github.com/KratosMultiphysics/Kratos/

@loumalouomega

Copy link
Copy Markdown
Author

Hello

@loumalouomega

Copy link
Copy Markdown
Author

Greetings

@codeplea

codeplea commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Thanks for the patch. Comparison as zIsGreater(a,b) functions overlaps with the logic branch, which does this as real operators, and the z-prefix naming is driven by the alphabetical lookup rather than by what's good to type. Closing, see CONTRIBUTING. TinyExpr is small enough that you can add this yourself in a fork. It only stays that way if I don't implement every request like this one.

@codeplea codeplea closed this Aug 5, 2026
@loumalouomega

Copy link
Copy Markdown
Author

OKay, I just wanted to contribute with the changes we applied in our system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants