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.

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

Simplify the data structure #16

Open
wants to merge 4 commits into
base: pcronin/performance-feature-branch
Choose a base branch
from

Conversation

PatrickCronin
Copy link
Owner

This PR is an attempt to implement @oschwald's idea. We'll simplify the data structure and see how benchmarks perform.

@coveralls
Copy link

coveralls commented Dec 22, 2021

Pull Request Test Coverage Report for Build 3810326162

  • 96 of 153 (62.75%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-14.8%) to 68.783%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/routesum/rstree/rstree.go 94 151 62.25%
Totals Coverage Status
Change from base Build 3810318927: -14.8%
Covered Lines: 260
Relevant Lines: 378

💛 - Coveralls

@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch 5 times, most recently from 9721e1e to b095e87 Compare December 28, 2021 05:38
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch from b095e87 to 9c88c2e Compare January 2, 2022 06:11
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch from 9c88c2e to 0f9ed1f Compare October 6, 2022 15:04
@PatrickCronin PatrickCronin changed the base branch from main to pcronin/performance-feature-branch October 6, 2022 15:04
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch 2 times, most recently from 0958b58 to f3065c3 Compare October 6, 2022 15:16
@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch from f6dc114 to b8782e3 Compare October 6, 2022 15:29
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch from f3065c3 to d9b0fb1 Compare October 6, 2022 16:23
@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch from 04aba48 to 101f1df Compare October 6, 2022 17:47
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch from d9b0fb1 to 35c4f06 Compare October 6, 2022 17:51
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch from 35c4f06 to 8e4c0a7 Compare October 20, 2022 15:21
@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch from 3191ad4 to 919d6ed Compare December 30, 2022 14:55
@PatrickCronin PatrickCronin force-pushed the pcronin/oschwalds-idea branch 2 times, most recently from 77a8f86 to 2128dd2 Compare December 30, 2022 17:08
@PatrickCronin PatrickCronin force-pushed the pcronin/performance-feature-branch branch 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