-
Notifications
You must be signed in to change notification settings - Fork 8
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
Term graph #12
Term graph #12
Conversation
Oooooh! 😍 I'll try to take a look at this tonight. Awesome work! |
@kderme How is the consistency of the graphs from step to step? If you're tracing the execution of a program, how much do the nodes jump around? |
They actually jump around much less than I expected. I have all results from examples here and a script to run all. A function can jump from bottom to top after it`s been applied (for example before and after) But in general there is a lot of stability. |
Just took a closer look, really cool stuff. Have been wanting to add this but never got around to it. I'm happy to merge this as-is, or do you want to address some of the points you mention above? If not, do you want to open some tickets about those so that we don't forget? One very minor niggle: typo: Nice! :) |
Happy you liked it. For now I can open some of the issues I mention above. In the future I may find some time to address them. |
Ok, great, merging as is :) |
@@ -1,10 +1,11 @@ | |||
module Main (main) where | |||
module Main (main) where |
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.
???
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.
oops..
Related Issue #2
This PR adds the option of visualizing the heap as a graph.
A step of examples/skim.hs:
This PR also adds a new example examples/graphs.hs
This is just a first attempt. There are a lot more thing that can be improved, like