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

Added pooling for path segments. #5066

Merged
merged 11 commits into from
May 17, 2022
Merged

Added pooling for path segments. #5066

merged 11 commits into from
May 17, 2022

Conversation

PascalSenn
Copy link
Member

Adds pool for path segments

|                      Method |         Mean |     Error |    StdDev |       Median | Rank |    Gen 0 | Gen 1 | Gen 2 | Allocated |
|---------------------------- |-------------:|----------:|----------:|-------------:|-----:|---------:|------:|------:|----------:|
|      V12_CreateNotManyPaths |     924.5 ns |   4.87 ns |   4.56 ns |     923.2 ns |    1 |   2.0847 |     - |     - |    4360 B |
|   Pooled_CreateNotManyPaths |   3,957.2 ns |  78.61 ns | 172.56 ns |   3,976.2 ns |    2 |        - |     - |     - |         - |
|    V12_CreateManyNamedPaths |  92,758.1 ns | 532.62 ns | 498.21 ns |  92,621.1 ns |    3 | 208.6182 |     - |     - |  436400 B |
| Pooled_CreateManyNamedPaths | 130,444.7 ns | 542.89 ns | 507.82 ns | 130,477.9 ns |    4 |        - |     - |     - |         - |

@PascalSenn PascalSenn changed the title DRAFT: Add pool for path segments WIP: Add pool for path segments May 15, 2022
@PascalSenn PascalSenn changed the title WIP: Add pool for path segments Add pool for path segments May 17, 2022
@michaelstaib michaelstaib changed the title Add pool for path segments Added pooling for path segments. May 17, 2022
@michaelstaib michaelstaib merged commit 72e410f into main May 17, 2022
@michaelstaib michaelstaib deleted the pse/add-name-path-pool branch May 17, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants