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

tx generator: CI benchmarks of the tx-generator #3726

Closed
wants to merge 2 commits into from

Conversation

MarcFontaine
Copy link
Contributor

No description provided.

@MarcFontaine MarcFontaine added enhancement New feature or request WIP Work In Progress (cannot be merged yet) labels Mar 29, 2022
@MarcFontaine MarcFontaine self-assigned this Mar 29, 2022
@MarcFontaine MarcFontaine force-pushed the mafo/tx-generator-ci-benchmark branch from a47b3ad to 6c70f09 Compare March 30, 2022 14:12
@MarcFontaine MarcFontaine removed the WIP Work In Progress (cannot be merged yet) label Mar 30, 2022
@MarcFontaine MarcFontaine marked this pull request as ready for review March 30, 2022 14:13
@MarcFontaine MarcFontaine force-pushed the mafo/tx-generator-ci-benchmark branch 3 times, most recently from 0e74304 to 48da5cf Compare April 7, 2022 11:11
@MarcFontaine MarcFontaine force-pushed the mafo/tx-generator-ci-benchmark branch 2 times, most recently from 326e357 to ab1d79b Compare April 13, 2022 09:57
For a while the tx-generator has been generating transaction just in time,
and the `TxSendQueue` had been used with a list of dummy transactions.
This PR removes the `TxSendQueue` altogether.
* Add a small built-in tx-generator-script.
* Add cabal-level criterion benchmark.
* Add new 'selftest' command: write some txs to a outfile.
@MarcFontaine MarcFontaine force-pushed the mafo/tx-generator-ci-benchmark branch from ab1d79b to 434cfe0 Compare April 19, 2022 07:46
iohk-bors bot added a commit that referenced this pull request Apr 19, 2022
3789: tx-generator: read path of node config file from CLI r=MarcFontaine a=MarcFontaine

This is a combination of several changes, that used to be separate PRs:
#3710
#3726
#3776

The only user-visible change is a new CLI option:
``Usage: tx-generator json_highlevel FILEPATH [-n|--nodeConfig FILENAME]``
 It allows one to set/overwrite the node config file.

Co-authored-by: MarcFontaine <MarcFontaine@users.noreply.github.com>
@MarcFontaine
Copy link
Contributor Author

These changes have been includes in #3789

@MarcFontaine MarcFontaine deleted the mafo/tx-generator-ci-benchmark branch May 31, 2022 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant