Skip to content
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

ExpToSegment not preserving right associativity with parens #1445

Open
7h3kk1d opened this issue Dec 19, 2024 · 2 comments
Open

ExpToSegment not preserving right associativity with parens #1445

7h3kk1d opened this issue Dec 19, 2024 · 2 comments
Assignees
Labels

Comments

@7h3kk1d
Copy link
Contributor

7h3kk1d commented Dec 19, 2024

Screenshot From 2024-12-19 11-49-01

Power (**) is right associative but this example is explicitly building up a left associative tree but ExpToSegment doesn't add parens in the left case.

@7h3kk1d 7h3kk1d added the bug label Dec 19, 2024
@cyrus-
Copy link
Member

cyrus- commented Dec 19, 2024

Is this specific to power or in general?

@7h3kk1d
Copy link
Contributor Author

7h3kk1d commented Dec 19, 2024

Is this specific to power or in general?

It's a general right associativity issue on binary operators. I came across it in the property test work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Team Editor
Development

No branches or pull requests

3 participants