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

Experimental Python Support #4

Merged
merged 47 commits into from
Sep 14, 2024
Merged

Experimental Python Support #4

merged 47 commits into from
Sep 14, 2024

Conversation

tmr232
Copy link
Owner

@tmr232 tmr232 commented Sep 14, 2024

Language Support

  • Add initial support for Python

Visualization

  • Add support for node clusters. This is used heavily in Python, for context-managers and exception-handling.

Demo

  • Add Python support
  • Add sharing - click the "Share" button to get a sharable link to what you currently see

Testing

  • Enable live-testing with the web viewer. Requires that you run both bun web-tests --watch and bun web at the same time.
  • By default, bun web only shows failing tests
  • bun web color-codes tests to note which are failing

Extension

  • No changes

Known Issues

  • Backlinks are no longer thicker than normal links. That said, they were half-broken to begin with and were somewhat arbitrary.

This caused clusters to be broken up in the resulting code, as we had multiple clusters with the same ID.
This is needed to fit with the try-return-finally flow
This is important when working with clusters, as it differentiates between the case
where the cluster has a return and when it does not.
…ry-complex.

This solves a lot of layout issues, as there is no need to link back into the containing cluster.
Additionally, it is more correct, as the 
eturn happens outside the original cluster and
does not have 	ry semantics.
This seems to work nicely, but the DOT layout is awful.
For every sample in the comment tests, we now generate a DOT file and compare it to a golden version.
We'll probably break this often, but I want to only break it when I expect it.
@tmr232 tmr232 merged commit c521670 into main Sep 14, 2024
1 check passed
@tmr232 tmr232 deleted the share-demo branch September 14, 2024 19:32
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.

1 participant