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

Add more granularity to data column proof computation metric #6371

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

jimmygchen
Copy link
Member

Issue Addressed

Add more granularity to data column proof computation metric to capture more variations between 0.25 and 1 second.

We currently get a lot of data point at 500ms and some at 1s, would be good to get more granularity there on computation time.

…re more variations between 0.25 and 1 second.
@jimmygchen jimmygchen added ready-for-review The code is ready for review low-hanging-fruit Easy to resolve, get it before someone else does! das Data Availability Sampling labels Sep 9, 2024
@jimmygchen jimmygchen added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Sep 9, 2024
@jimmygchen
Copy link
Member Author

@mergify queue

Copy link

mergify bot commented Sep 9, 2024

queue

🛑 Command queue cancelled because of a new queue command with different arguments

@jimmygchen
Copy link
Member Author

@mergify dequeue

Copy link

mergify bot commented Sep 9, 2024

dequeue

✅ The pull request has been removed from the queue default

@jimmygchen
Copy link
Member Author

@mergify queue

Copy link

mergify bot commented Sep 9, 2024

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 815567a

@mergify mergify bot merged commit 815567a into sigp:unstable Sep 9, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
das Data Availability Sampling low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants