Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

wrap caps allocator to preempt memory allocation errors #130

Merged
merged 5 commits into from
Oct 23, 2024
Merged

Conversation

Iamdavidonuh
Copy link
Collaborator

@Iamdavidonuh Iamdavidonuh commented Oct 22, 2024

Part of #100

@Iamdavidonuh Iamdavidonuh requested a review from deven96 October 22, 2024 13:42
Copy link

github-actions bot commented Oct 22, 2024

Test Results

137 tests   137 ✅  3m 30s ⏱️
  8 suites    0 💤
  2 files      0 ❌

Results for commit f8b3bba.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Oct 22, 2024

Benchmark Results

group                                                        main                                   pr
-----                                                        ----                                   --
store_batch_insertion_without_predicates/size_100            1.00    836.0±2.66µs        ? ?/sec    1.00    833.1±8.22µs        ? ?/sec
store_batch_insertion_without_predicates/size_1000           1.01      9.5±0.05ms        ? ?/sec    1.00      9.5±0.07ms        ? ?/sec
store_batch_insertion_without_predicates/size_10000          1.00     98.0±0.33ms        ? ?/sec    1.00     97.7±0.64ms        ? ?/sec
store_batch_insertion_without_predicates/size_100000         1.03   1007.1±9.37ms        ? ?/sec    1.00    974.1±5.35ms        ? ?/sec
store_retrieval_no_condition/size_100                        1.04      2.8±0.04ms        ? ?/sec    1.00      2.7±0.03ms        ? ?/sec
store_retrieval_no_condition/size_1000                       1.04     20.5±0.25ms        ? ?/sec    1.00     19.7±0.29ms        ? ?/sec
store_retrieval_no_condition/size_10000                      1.04    203.3±1.30ms        ? ?/sec    1.00    195.0±2.77ms        ? ?/sec
store_retrieval_no_condition/size_100000                     1.03       2.0±0.03s        ? ?/sec    1.00  1957.4±15.40ms        ? ?/sec
store_retrieval_non_linear_kdtree/size_100                   1.05      3.0±0.04ms        ? ?/sec    1.00      2.9±0.04ms        ? ?/sec
store_retrieval_non_linear_kdtree/size_1000                  1.04     22.3±0.28ms        ? ?/sec    1.00     21.3±0.14ms        ? ?/sec
store_retrieval_non_linear_kdtree/size_10000                 1.04    218.5±1.66ms        ? ?/sec    1.00    210.5±1.91ms        ? ?/sec
store_retrieval_non_linear_kdtree/size_100000                1.04       2.2±0.03s        ? ?/sec    1.00       2.1±0.01s        ? ?/sec
store_sequential_insertion_without_predicates/size_100       1.10  1861.8±29.30µs        ? ?/sec    1.00  1697.1±22.51µs        ? ?/sec
store_sequential_insertion_without_predicates/size_1000      1.02     18.6±0.50ms        ? ?/sec    1.00     18.2±0.18ms        ? ?/sec
store_sequential_insertion_without_predicates/size_10000     1.05    188.8±2.35ms        ? ?/sec    1.00    179.1±2.52ms        ? ?/sec
store_sequential_insertion_without_predicates/size_100000    1.05  1899.1±48.34ms        ? ?/sec    1.00  1807.5±29.28ms        ? ?/sec

@Iamdavidonuh Iamdavidonuh marked this pull request as ready for review October 22, 2024 21:55
@deven96 deven96 merged commit cf5c84d into main Oct 23, 2024
5 checks passed
@deven96 deven96 deleted the custom_alloc branch November 27, 2024 16:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants