Skip to content

Conversation

@asglover
Copy link
Collaborator

This PR, originally only supposed to contain visualization code and examples, expanded include factoring out serialized egraph into its own module, and factoring out some egglog utilities to make constructing the visualization example (and future compiler work) more straightforward. It is a lot of changes, I'm happy to refactor them further, or break this into multiple PR so it's easier to review.

@asglover asglover requested a review from jafioti December 24, 2025 05:19
austin_glover added 2 commits December 24, 2025 05:27
Copy link
Collaborator

@jafioti jafioti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, great job! been meaning to factor out the egglog file, good you got around to it. also good catch with some of the import cleanups re: fxhashmap.

@jafioti jafioti marked this pull request as ready for review December 25, 2025 03:18
@jafioti jafioti merged commit e0727a3 into main Dec 25, 2025
5 checks passed
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.

3 participants