-
Notifications
You must be signed in to change notification settings - Fork 16
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
Port to new pymbolic #242
Port to new pymbolic #242
Conversation
This seems to be in a better state than expected! The remaining deprecation warnings are from |
1eee3ac
to
623eb93
Compare
@inducer This should be good for a look now that all the upstream changes are in. |
I have some changes based on inducer/pymbolic#159 that I'd like to push. I imagine you have some changes based on inducer/pymbolic#160 that you'd like to push. How do we sort them to avoid stomping on one another's feet? |
I haven't actually started making any changes here based on inducer/pymbolic#160, so go right ahead with yours! This will start complaining once inducer/pymbolic#160 is merged though 😁 |
42a5cf8
to
ec3bdc4
Compare
Uh-oh. Better get this in quickly. :) |
fdd934a
to
707dec3
Compare
2140d25
to
579f0c2
Compare
Updates things to use
expr_dataclass
and fixes various deprecation warnings.Needs: