Skip to content

chore: capture RPC calls + correct timings (next cherry-pick)#14668

Merged
Thunkar merged 2 commits intonextfrom
gj/timings_into_next
Jun 2, 2025
Merged

chore: capture RPC calls + correct timings (next cherry-pick)#14668
Thunkar merged 2 commits intonextfrom
gj/timings_into_next

Conversation

@Thunkar
Copy link
Copy Markdown
Contributor

@Thunkar Thunkar commented Jun 2, 2025

Fixes merge conflicts. next version of #14633

This PR allows us to run simulation with an instrumented node, that we
can hijack to capture timing information on RPC calls. It also addresses
a bug that made us measure witgen time incorrectly (and incidentally
caused the unnacounted time to sometimes go negative)

---------

Co-authored-by: Nicolás Venturo <nicolas.venturo@gmail.com>
@Thunkar Thunkar requested a review from benesjan June 2, 2025 07:20
@Thunkar Thunkar self-assigned this Jun 2, 2025
@Thunkar Thunkar added this pull request to the merge queue Jun 2, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 2, 2025
@Thunkar Thunkar added this pull request to the merge queue Jun 2, 2025
Merged via the queue into next with commit 4e61e78 Jun 2, 2025
4 checks passed
@Thunkar Thunkar deleted the gj/timings_into_next branch June 2, 2025 08:57
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.

2 participants