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.

Sign up for GitHub

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

Jump to bottom

Revert "Revert "feat(timeit): add function timing decorator (#118)"" #120

Merged

wpbonelli merged 3 commits into develop from revert-119-develop

Sep 29, 2023

Contributor

wpbonelli commented Sep 29, 2023 •

edited

Loading

reverts Revert "feat(timeit): add function timing decorator (#118)" #119, pardon the indecision
seems justified if devtools is used in notebooks as proposed in Self-contained example notebooks modflow6#1348
rename to @timed / timed() to avoid collision with builtin timeit module/function
use timeit internally instead of manually timing

wpbonelli and others added 2 commits

September 29, 2023 08:38


          Revert "Revert "feat(timeit): add function timing decorator (#118)" (#…

d47f5cf

…119)"

This reverts commit d725ddb.


          rename timeit -> timed, use builtin timeit internally

c495cd8

wpbonelli marked this pull request as ready for review

September 29, 2023 19:49


          update docs

dc91479

wpbonelli merged commit 2f4cdf2 into develop

0 of 3 checks passed

wpbonelli deleted the revert-119-develop branch

September 29, 2023 21:03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet