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.

Sign up for GitHub

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

Jump to bottom

Expand or Move away fromParametricExpression #1597

Open
erinzmoore opened this issue Sep 11, 2024 · 0 comments
Open

Expand or Move away fromParametricExpression #1597

erinzmoore opened this issue Sep 11, 2024 · 0 comments
Labels
dslx DSLX (domain specific language) implementation / front-end
Milestone
DSLX large projects

Comments

@erinzmoore
Copy link
Collaborator

erinzmoore commented Sep 11, 2024 •
edited
Loading

Some parametric entities rely on ParametricExpression, which currently does not support all operations that should be allowable for parametric expressions. Consider migrating away from ParametricExpression and just using Expr directly in these cases. Alternatively, refactor ParametricExpression to be more flexible/expandable.
@erinzmoore erinzmoore added the dslx DSLX (domain specific language) implementation / front-end label Sep 11, 2024
@erinzmoore erinzmoore added this to the DSLX large projects milestone Sep 11, 2024
@erinzmoore erinzmoore mentioned this issue Sep 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dslx DSLX (domain specific language) implementation / front-end
Projects
XLS usability
Status: No status
Milestone
DSLX large projects
Development

No branches or pull requests
1 participant
@erinzmoore