[Docs] Updating benchmarkmarking.md with an updated sample output #12187
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
Member
|
Could also mention that a help message can be printed by |
jeffrade
force-pushed
the
benchmark_output
branch
from
3a62e59 to
7eaedea
Compare
Contributor
Author
|
@MarcoFalke Added a help section for the |
Member
|
ACK 7eaedea53214da7bcfc51b794b54de3663f02e03 |
jeffrade
force-pushed
the
benchmark_output
branch
from
7eaedea to
b21244e
Compare
Member
|
utACK b21244e |
maflcko
pushed a commit
that referenced
this pull request
Jan 22, 2018
…ple output b21244e Updating benchmarkmarking.md with an updated sample output and help options (Jeff Rade) Pull request description: This PR is just a documentation update for someone (or myself) that looks into finishing up #7883 in the future. Looked through #7883 and appears [ryanofsky's PR](#8873) setup the benchmarks, but there are `FIXME` comments to pull in data from `test/` to get a larger data set (assuming reason why 7883 is still open). Tree-SHA512: d758efc659c75f2b3ceb376f5a466c4234354077e4671ac3eb901c082c4e519ce5ff592cea4742711050b4ce56a1b65ef69433dd74e7db3eb11a8567d517d9e2
PastaPastaPasta
pushed a commit
to PastaPastaPasta/dash
that referenced
this pull request
Apr 5, 2020
…ted sample output b21244e Updating benchmarkmarking.md with an updated sample output and help options (Jeff Rade) Pull request description: This PR is just a documentation update for someone (or myself) that looks into finishing up bitcoin#7883 in the future. Looked through bitcoin#7883 and appears [ryanofsky's PR](bitcoin#8873) setup the benchmarks, but there are `FIXME` comments to pull in data from `test/` to get a larger data set (assuming reason why 7883 is still open). Tree-SHA512: d758efc659c75f2b3ceb376f5a466c4234354077e4671ac3eb901c082c4e519ce5ff592cea4742711050b4ce56a1b65ef69433dd74e7db3eb11a8567d517d9e2
jasonbcox
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Oct 31, 2020
…sample output Summary: b21244e Updating benchmarkmarking.md with an updated sample output and help options (Jeff Rade) Pull request description: This PR is just a documentation update for someone (or myself) that looks into finishing up #7883 in the future. Looked through #7883 and appears [ryanofsky's PR](bitcoin/bitcoin#8873) setup the benchmarks, but there are `FIXME` comments to pull in data from `test/` to get a larger data set (assuming reason why 7883 is still open). --- Backport of Core [[bitcoin/bitcoin#12187 | PR12187]] Test Plan: read it Reviewers: #bitcoin_abc, PiRK Reviewed By: PiRK Differential Revision: https://reviews.bitcoinabc.org/D8205
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
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.
This PR is just a documentation update for someone (or myself) that looks into finishing up #7883 in the future.
Looked through #7883 and appears ryanofsky's PR setup the benchmarks, but there are
FIXMEcomments to pull in data fromtest/to get a larger data set (assuming reason why 7883 is still open).