Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

updating benchmarking script #1012

Merged
merged 1 commit into from
Feb 21, 2022
Merged

updating benchmarking script #1012

merged 1 commit into from
Feb 21, 2022
+6 −6

Conversation

gilescope
Copy link
Contributor

we should be using production profile for benchmarks.
--raw does not exist: paritytech/substrate#10771

@gilescope
updating benchmarking script
753f269 
we should be using production profile for benchmarks.
--raw does not exist: paritytech/substrate#10771
@gilescope gilescope added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low 📌 labels Feb 21, 2022
@gilescope gilescope merged commit 8cf65b2 into master Feb 21, 2022
@gilescope gilescope deleted the giles-raw-now-json branch February 21, 2022 15:18
@ggwpez
Copy link
Member

ggwpez commented Feb 21, 2022 •
edited
Loading

Do you actually use the stdout json? If you want it into a file you can use --json-file=path instead.
@gilescope

PS: Using the --json flag will now not print a summary anymore.

@gilescope gilescope mentioned this pull request Mar 14, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@NachoPal NachoPal NachoPal approved these changes

@hbulgarini hbulgarini Awaiting requested review from hbulgarini

Assignees
No one assigned
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gilescope @ggwpez @NachoPal