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

feat: add incremental benchmarks #53

Merged
merged 7 commits into from
Dec 10, 2023
Merged

Conversation

fspoettel
Copy link
Owner

closes #45

src/template/commands/time.rs Outdated Show resolved Hide resolved
src/template/run_multi.rs Outdated Show resolved Hide resolved
src/main.rs Outdated Show resolved Hide resolved
src/template/timings.rs Outdated Show resolved Hide resolved
src/template/timings.rs Outdated Show resolved Hide resolved
@fspoettel fspoettel force-pushed the feat/incremental-timings branch 3 times, most recently from cb0d0fb to 2ef57a8 Compare December 10, 2023 12:21
Co-authored-by: Tristan Guichaoua <33934311+tguichaoua@users.noreply.github.com>
@fspoettel fspoettel force-pushed the feat/incremental-timings branch from 2ef57a8 to 0043528 Compare December 10, 2023 12:21
@fspoettel
Copy link
Owner Author

thanks for the helpful review @tguichaoua, this should be ready now.

Copy link
Contributor

@tguichaoua tguichaoua left a comment

Choose a reason for hiding this comment

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

Just one last change 😉
The rest LGTM

src/template/commands/time.rs Outdated Show resolved Hide resolved
fspoettel and others added 2 commits December 10, 2023 13:51
Co-authored-by: Tristan Guichaoua <33934311+tguichaoua@users.noreply.github.com>
@fspoettel fspoettel merged commit 874f57b into main Dec 10, 2023
1 check passed
@fspoettel fspoettel deleted the feat/incremental-timings branch December 10, 2023 12:55
tguichaoua added a commit to tguichaoua/advent-of-code-2023-rust that referenced this pull request Dec 11, 2023
# 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.

Time individual puzzles
2 participants