Skip to content

[Cadence VM] Improve compiler/VM / interpreter comparison tool #8221

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.

Sign up for GitHub

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

Jump to bottom
Open
turbolent wants to merge 5 commits into
base: feature/cadence-vm
Choose a base branch
from
Open

[Cadence VM] Improve compiler/VM / interpreter comparison tool #8221

turbolent wants to merge 5 commits into from

Conversation

@turbolent
Copy link
Member

@turbolent turbolent commented Dec 3, 2025 •
edited
Loading

  • Add progress logging
  • Add support for subscribing to new blocks and comparing them as they are produced

@turbolent turbolent requested review from a team December 3, 2025 20:43
@turbolent turbolent requested a review from a team as a code owner December 3, 2025 20:43
@codecov-commenter
Copy link

codecov-commenter commented Dec 3, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 0% with 145 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cmd/util/cmd/compare-cadence-vm/cmd.go 0.00% 88 Missing ⚠️
utils/debug/api.go 0.00% 32 Missing ⚠️
cmd/util/cmd/debug-tx/cmd.go 0.00% 25 Missing ⚠️

📢 Thoughts on this report? Let us know!

SupunS
SupunS reviewed Dec 3, 2025
View reviewed changes
@turbolent turbolent requested a review from SupunS December 4, 2025 00:02
@turbolent turbolent changed the title [Cadence VM] Add progress logging to comparison tool [Cadence VM] Improve compiler/VM / interpreter comparison tool Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SupunS SupunS Awaiting requested review from SupunS

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@turbolent @codecov-commenter @SupunS