Skip to content

Performance Feature Branch#34

Open
PatrickCronin wants to merge 1 commit intomainfrom
pcronin/performance-feature-branch
Open

Performance Feature Branch#34
PatrickCronin wants to merge 1 commit intomainfrom
pcronin/performance-feature-branch

Conversation

@PatrickCronin
Copy link
Owner

Over the course of making routesum faster, we've tried a number of things, and added a few features that are great for monitoring both performance and memory usage. Up to this point we've been merging these straight into main. In retrospect, some of these changes are useful only temporarily, and so don't really belong in main. This PR (well, this branch really) will store the useful changes that will support the performance and memory monitoring, and can be the branch upon which all current performance tweaks can be based on, before selecting the winning branch, rebasing it on main, and merging that.

@coveralls
Copy link

coveralls commented Oct 6, 2022

Pull Request Test Coverage Report for Build 3921274111

  • 5 of 39 (12.82%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-9.03%) to 80.864%

Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/routesum/main.go 5 39 12.82%
Totals Coverage Status
Change from base Build 3810317325: -9.03%
Covered Lines: 262
Relevant Lines: 324

💛 - Coveralls

@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch 4 times, most recently from 04aba48 to 101f1df Compare October 6, 2022 17:47
@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch 2 times, most recently from 919d6ed to 1bb8216 Compare December 31, 2022 02:59
@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch from ac52555 to 8f16126 Compare January 15, 2023 01:57
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.

2 participants