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.

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

Speeds up module.add_debug_info and add_metadata by 2x #887

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Speeds up module.add_debug_info and add_metadata by 2x #887

wants to merge 11 commits into from
+147 −24

Commits on Oct 31, 2022

  1. Reduces repeated sorting and hashing to speed up adding debug info.

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    84812fa View commit details
    Browse the repository at this point in the history

  2. Making adjustments to make module run faster

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    1edf17d View commit details
    Browse the repository at this point in the history

  3. Adjusting test parameters

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    2b9dc33 View commit details
    Browse the repository at this point in the history

  4. Simplifying test

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    e5585c7 View commit details
    Browse the repository at this point in the history

  5. Tests are showing 15% faster at adding debug info

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    1032046 View commit details
    Browse the repository at this point in the history

  6. Fixed a bug

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    6e4590c View commit details
    Browse the repository at this point in the history

  7. Adding a cache on the hashes

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    0281979 View commit details
    Browse the repository at this point in the history

  8. Benchmark now shows it takes 50% less time

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    bffb1f3 View commit details
    Browse the repository at this point in the history

  9. Turning off print message

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    c5ad886 View commit details
    Browse the repository at this point in the history

  10. Remove profiler and always print the performance information

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    4960a20 View commit details
    Browse the repository at this point in the history

  11. Fixes flake8 errors

    @bslatkin
    bslatkin committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    7f3392d View commit details
    Browse the repository at this point in the history