Skip to content

feature: Allow long traces that don't cross any other traces#74

Merged
seveibar merged 7 commits intotscircuit:mainfrom
0hmX:1
Oct 8, 2025
Merged

feature: Allow long traces that don't cross any other traces#74
seveibar merged 7 commits intotscircuit:mainfrom
0hmX:1

Conversation

@0hmX
Copy link
Copy Markdown
Contributor

@0hmX 0hmX commented Oct 8, 2025

close: #68
claim #68

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 8, 2025

@0hmX is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

@0hmX 0hmX force-pushed the 1 branch 2 times, most recently from 840566b to c80dcf6 Compare October 8, 2025 04:35
Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

This is pretty good, i think with some tweaks this will be accepted.

I would add the example as a snapshot test as well (I didn't see the snapshot, only the page) it'd be nice to make sure it works in a snapshot test

nice work

@0hmX 0hmX force-pushed the 1 branch 2 times, most recently from de5e484 to e29c025 Compare October 8, 2025 06:47
@vercel
Copy link
Copy Markdown

vercel bot commented Oct 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
schematic-trace-solver Ready Ready Preview Comment Oct 8, 2025 2:59pm

@seveibar seveibar merged commit fd29717 into tscircuit:main Oct 8, 2025
3 of 4 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.

Allow long traces that don't cross any other traces

2 participants