Print total tasks started on benchmark #3659
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Mar 26, 2025
Closed
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Mar 26, 2025
Merged
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
b673d39 to
c7d6782
Compare
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
from
62f0dcc to
d1d9233
Compare
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
c7d6782 to
c90012d
Compare
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
from
d1d9233 to
12d522a
Compare
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
from
12d522a to
ce058c8
Compare
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
c90012d to
ad554e9
Compare
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
ad554e9 to
c5b66eb
Compare
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
2 times, most recently
from
c28c907 to
2e91a3a
Compare
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
c5b66eb to
9fff016
Compare
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
from
2e91a3a to
89bcc44
Compare
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
9fff016 to
f1a6883
Compare
This was referenced Apr 8, 2025
ma2bd
approved these changes
Apr 9, 2025
ndr-ds
force-pushed
the
03-26-more_dashboard_changes
branch
from
f1a6883 to
b16f4be
Compare
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
from
89bcc44 to
c68aac3
Compare
ndr-ds
changed the base branch from
03-26-more_dashboard_changes
to
graphite-base/3659
ndr-ds
force-pushed
the
03-26-print_total_tasks_started_on_benchmark
branch
from
c68aac3 to
e91ebfa
Compare
This was referenced Apr 11, 2025
Closed
Merged
ma2bd
pushed a commit
that referenced
this pull request
Apr 14, 2025
## Motivation Sometimes some of the tasks will lag behind for some reason and take longer to start. ## Proposal Have to investigate that, but for now let's log some information so we know not all tasks have started yet. Specially when dealing with a lot of chains, this is useful. ## Test Plan Run it, see messages printed ## Release Plan - Nothing to do / These changes follow the usual release cycle.
This was referenced Apr 14, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
Sometimes some of the tasks will lag behind for some reason and take longer to start.
Proposal
Have to investigate that, but for now let's log some information so we know not all tasks have started yet. Specially when dealing with a lot of chains, this is useful.
Test Plan
Run it, see messages printed
Release Plan