-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Specification of math expressions #579
Conversation
Related to PEtab-dev/PEtab#579 * ANTLR grammar for PEtab math expressions + derived lexer/parser/visitor * Test cases * PEtab -> SymPy converter
Related to PEtab-dev/PEtab#579 * ANTLR grammar for PEtab math expressions + derived lexer/parser/visitor * Test cases * PEtab -> SymPy converter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Co-authored-by: Dilan Pathirana <59329744+dilpath@users.noreply.github.com>
Co-authored-by: Dilan Pathirana <59329744+dilpath@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am happy with all proposed changes, and think that the current syntax encapsulates the relevant functions.
As a potential addition, in the test-cases I just think three tests on Boolean <-> float conversion could be added (when everyone agree on that point).
I'm happy with the proposed description. If anything I'd rename the |
Related to PEtab-dev/PEtab#579 * ANTLR grammar for PEtab math expressions + derived lexer/parser/visitor * Test cases * PEtab -> SymPy converter Note: All sympy symbols in PEtab match expressions now have `real=True`.
Add math expression test cases. Related to PEtab-dev/PEtab#579
Previously, the math expression syntax wasn't specified. This was very problematic, because different libraries and programming languages have different names for the same functions, and more importantly, differ in operator precedence. Co-authored-by: Dilan Pathirana <59329744+dilpath@users.noreply.github.com> Co-authored-by: dilpath <dilan.private+github@outlook.com>
Previously, the math expression syntax wasn't specified. This was very problematic, because different libraries and programming languages have different names for the same functions, and more importantly, differ in operator precedence. Co-authored-by: Dilan Pathirana <59329744+dilpath@users.noreply.github.com> Co-authored-by: dilpath <dilan.private+github@outlook.com>
Previously, the math expression syntax wasn't specified. This was very problematic, because different libraries and programming languages have different names for the same functions, and more importantly, differ in operator precedence.
👀 https://petab--579.org.readthedocs.build/en/579/documentation_data_format.html#math-expressions-syntax
Examples / test cases: https://github.com/dweindl/libpetab-python/blob/parse_math/tests/math/math_tests.yaml