Skip to content

feat: adding a simple benchmarking CLI for the examples#33

Merged
Dominik1999 merged 1 commit intomainfrom
add_benchmarking_cli
Dec 20, 2022
Merged

feat: adding a simple benchmarking CLI for the examples#33
Dominik1999 merged 1 commit intomainfrom
add_benchmarking_cli

Conversation

@Dominik1999
Copy link
Collaborator

Closes #9

@Dominik1999 Dominik1999 force-pushed the add_benchmarking_cli branch 2 times, most recently from 8e390d1 to 14d9d40 Compare December 19, 2022 18:28
Copy link

@itzmeanjan itzmeanjan left a comment

Choose a reason for hiding this comment

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

Looks good.

@Dominik1999 Dominik1999 force-pushed the add_benchmarking_cli branch 3 times, most recently from eb311e1 to 5cf1e09 Compare December 20, 2022 08:27
@Dominik1999 Dominik1999 removed the request for review from bobbinth December 20, 2022 10:34
@Dominik1999 Dominik1999 force-pushed the add_benchmarking_cli branch 2 times, most recently from fad03aa to 09d96ec Compare December 20, 2022 12:57
Copy link

@itzmeanjan itzmeanjan left a comment

Choose a reason for hiding this comment

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

Looks good !

@Dominik1999 Dominik1999 requested a review from grjte December 20, 2022 13:28
@itzmeanjan
Copy link

@Dominik1999 Issuing cargo fmt should resolve issues related to source formatting.

Copy link
Contributor

@grjte grjte left a comment

Choose a reason for hiding this comment

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

Looks great, thank you! There are just a couple small things I'd prefer to adjust before merging.

@Dominik1999 Dominik1999 requested a review from grjte December 20, 2022 15:29
Copy link
Contributor

@grjte grjte left a comment

Choose a reason for hiding this comment

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

Looks good, thanks @Dominik1999! I left one comment inline - maybe you can make an issue to address this in the future?

@Dominik1999 Dominik1999 merged commit a3e22dd into main Dec 20, 2022
@Dominik1999 Dominik1999 deleted the add_benchmarking_cli branch December 21, 2022 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create a small benchmarking CLI

3 participants