Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Run omni-bencher overhead cmd in CI #6347

Open
ggwpez opened this issue Nov 4, 2024 · 2 comments · May be fixed by #7459
Open

Run omni-bencher overhead cmd in CI #6347

ggwpez opened this issue Nov 4, 2024 · 2 comments · May be fixed by #7459
Labels
D0-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder. T10-tests This PR/Issue is related to tests.

Comments

@ggwpez
Copy link
Member

ggwpez commented Nov 4, 2024

We need to ensure the new benchmark overhead command keeps working: #5891
Currently there is a small integration test in the CI, but we should have one like for the pallet benchmarking where we run it for all runtimes in this repo and then lateron also in the fellowship runtimes repo.
@ggwpez ggwpez added T10-tests This PR/Issue is related to tests. D0-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder. labels Nov 4, 2024
@clangenb
Copy link
Contributor

I would like to take on this one. I believe, even if it is a CI change, there is no issue with me not being a maintainer here?

@clangenb clangenb linked a pull request Feb 5, 2025 that will close this issue
Open
@clangenb
Copy link
Contributor

clangenb commented Feb 5, 2025

Started working on this, please assign to me. :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
D0-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder. T10-tests This PR/Issue is related to tests.
Projects
Runtime / FRAME
Status: Backlog
Milestone
No milestone
2 participants
@ggwpez @clangenb