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

Begin porting latency benchmark to shelley #1829

Merged
merged 2 commits into from
Jul 7, 2020
Merged

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Jun 30, 2020

Relates to #1825

Overview

These refactors are the start of porting the latency bench from the byron backend to the shelley backend.

The measurement and reporting code is split into a separate module.

The bench scenario functions are reworked so that swapping backend will be easier.

@rvl rvl self-assigned this Jun 30, 2020
@KtorZ KtorZ added the IMPROVEMENT Mark a PR as an improvement, for auto-generated CHANGELOG label Jun 30, 2020
@rvl rvl force-pushed the rvl/1825/estimate-fee branch 2 times, most recently from 30e730a to 47cbb3c Compare July 1, 2020 06:04
@rvl rvl changed the title Port latency benchmark to shelley Begin porting latency benchmark to shelley Jul 1, 2020
@rvl rvl force-pushed the rvl/1825/estimate-fee branch from 47cbb3c to cebd0b5 Compare July 1, 2020 06:12
@rvl rvl marked this pull request as ready for review July 1, 2020 06:13
@rvl rvl force-pushed the rvl/1825/estimate-fee branch from cebd0b5 to 4fc212f Compare July 1, 2020 06:15
@rvl rvl force-pushed the rvl/1825/estimate-fee branch from da4f2ea to f0f8e6a Compare July 2, 2020 06:24
This refactor begins porting of the latency bench to shelley.
@rvl rvl force-pushed the rvl/1825/estimate-fee branch from f0f8e6a to 414ba03 Compare July 2, 2020 07:10
@rvl rvl requested a review from paweljakubas July 3, 2020 12:27
Copy link
Contributor

@piotr-iohk piotr-iohk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@piotr-iohk
Copy link
Contributor

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jul 7, 2020

@iohk-bors iohk-bors bot merged commit e855495 into master Jul 7, 2020
@iohk-bors iohk-bors bot deleted the rvl/1825/estimate-fee branch July 7, 2020 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IMPROVEMENT Mark a PR as an improvement, for auto-generated CHANGELOG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants