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

Fix CI metrics_capture #1215

Merged
merged 1 commit into from
Jul 22, 2020
Merged

Fix CI metrics_capture #1215

merged 1 commit into from
Jul 22, 2020

Conversation

cryptonemo
Copy link
Collaborator

feat: enable proof validate by default on merkleproof bench

While proofs_count is CLI configurable, if it's more than the number
of nodes in the tree, just sequentially challenge each node in the
tree instead of randomly selecting and over-proving.

@cryptonemo cryptonemo force-pushed the metrics-capture branch 4 times, most recently from 5ded29b to 686391b Compare July 20, 2020 15:27
Make sure cargo is installed on remote machine.
Automatically add remote ssh key to known hosts.
Replace remaining election post references with window

feat: enable proof validate by default on merkleproof bench

While proofs_count is CLI configurable, if it's more than the number
of nodes in the tree, just sequentially challenge each node in the
tree instead of randomly selecting and over-proving.
@cryptonemo cryptonemo changed the title fix: enable metrics_capture in order to debug fix: CI metrics_capture Jul 22, 2020
@cryptonemo cryptonemo changed the title fix: CI metrics_capture Fix CI metrics_capture Jul 22, 2020
@cryptonemo cryptonemo merged commit 9be0a74 into master Jul 22, 2020
@cryptonemo cryptonemo deleted the metrics-capture branch October 26, 2020 16:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants