Skip to content

Conversation

@cdc-as81
Copy link
Collaborator

@cdc-as81 cdc-as81 commented Jan 6, 2026

No description provided.

@cdc-as81 cdc-as81 linked an issue Jan 6, 2026 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Benchmark Results

Hyperfine

  Group                  Bench                     Mean [ms]  Min [ms]  Max [ms]       Relative
  ---------------------  ---------------------  ------------  --------  --------  -------------
  periodic_counts_bench  no_periodic_reports       7.9 ± 1.5       7.1      12.0           1.00
  periodic_counts_bench  with_periodic_reports    12.6 ± 0.5      12.3      15.8    1.59 ± 0.32
  large_sir              baseline                  2.8 ± 0.0       2.8       2.9           1.00
  large_sir              ixa                     137.2 ± 2.1     134.7     147.9   48.74 ± 1.02
  large_sir              ixa_no_queries           36.3 ± 1.2      34.5      41.6   12.90 ± 0.47

Criterion

Regressions:
  Group          Bench                                                          Change  CI Lower  CI Upper
  -------------  -----------------------------------------------------------  --------  --------  --------
  examples       example-births-deaths                                          0.850%    0.538%    1.178%
  indexing       with_query_results_indexed_multi-property                      1.119%    0.864%    1.492%
  indexing       query_people_count_single_indexed_property                    11.443%   10.140%   12.639%
  indexing       with_query_results_single_indexed_property                     0.614%    0.098%    1.156%
  indexing       query_people_single_indexed_property                           1.151%    0.518%    1.906%
  sample_people  sampling_single_known_length                                   0.459%    0.164%    0.780%
  large_dataset  bench_query_population_derived_property                        1.006%    0.878%    1.144%
  large_dataset  bench_query_population_indexed_property                        4.006%    3.617%    4.271%

Improvements:
  Group          Bench                                                          Change  CI Lower  CI Upper
  -------------  -----------------------------------------------------------  --------  --------  --------
  examples       example-basic-infection                                       -4.053%   -5.076%   -2.700%
  indexing       query_people_multiple_individually_indexed_properties         -0.209%   -0.281%   -0.140%
  indexing       with_query_results_multiple_individually_indexed_properties   -4.069%   -4.964%   -3.248%
  indexing       query_people_indexed_multi-property                          -10.546%  -11.211%   -9.808%
  indexing       query_people_count_multiple_individually_indexed_properties   -4.897%   -5.960%   -3.761%
  sample_people  sampling_multiple_l_reservoir                                 -0.340%   -0.419%   -0.256%
  sample_people  sampling_multiple_known_length                                -1.110%   -1.380%   -0.855%
  large_dataset  bench_query_population_property                               -0.216%   -0.358%   -0.058%
  large_dataset  bench_query_population_multi_unindexed                        -0.812%   -1.187%   -0.413%

Unchanged:
  Group          Bench                                                          Change  CI Lower  CI Upper
  -------------  -----------------------------------------------------------  --------  --------  --------
  indexing       query_people_count_indexed_multi-property                     -0.255%   -0.662%    0.196%
  sample_people  sampling_single_l_reservoir                                    0.090%   -0.002%    0.166%
  large_dataset  bench_query_population_multi_indexed                           0.062%   -0.140%    0.240%

Copy link
Collaborator

@RobertJacobsonCDC RobertJacobsonCDC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes a lot of sense to me. I wouldn't bother with the comment:

// ProfilingContextExt moved to reports.rs

But that's just personal preference.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Benchmark Results

Hyperfine

  Group                  Bench                     Mean [ms]  Min [ms]  Max [ms]       Relative
  ---------------------  ---------------------  ------------  --------  --------  -------------
  large_sir              baseline                  2.9 ± 0.0       2.8       3.0           1.00
  large_sir              ixa                     137.2 ± 1.5     135.3     142.4   47.98 ± 0.89
  large_sir              ixa_no_queries           36.4 ± 0.8      34.6      38.4   12.71 ± 0.35
  periodic_counts_bench  no_periodic_reports       7.4 ± 0.2       7.2       8.1           1.00
  periodic_counts_bench  with_periodic_reports    12.8 ± 0.5      12.5      15.8    1.75 ± 0.08

Criterion

Regressions:
  Group          Bench                                                          Change  CI Lower  CI Upper
  -------------  -----------------------------------------------------------  --------  --------  --------
  indexing       with_query_results_indexed_multi-property                      2.286%    2.097%    2.471%
  indexing       with_query_results_single_indexed_property                     1.683%    1.284%    2.153%
  indexing       query_people_count_multiple_individually_indexed_properties    0.221%    0.100%    0.324%
  large_dataset  bench_query_population_derived_property                        0.541%    0.016%    1.104%
  large_dataset  bench_query_population_indexed_property                        2.322%    2.091%    2.541%

Improvements:
  Group          Bench                                                          Change  CI Lower  CI Upper
  -------------  -----------------------------------------------------------  --------  --------  --------
  examples       example-basic-infection                                       -5.778%   -6.368%   -5.047%
  indexing       query_people_count_single_indexed_property                   -12.360%  -13.435%  -11.192%
  indexing       query_people_single_indexed_property                          -4.990%   -6.620%   -3.268%
  indexing       query_people_indexed_multi-property                           -9.439%  -10.015%   -9.046%
  sample_people  sampling_multiple_l_reservoir                                 -0.445%   -0.798%   -0.181%
  sample_people  sampling_multiple_known_length                                -0.478%   -1.016%   -0.022%
  sample_people  sampling_single_known_length                                  -0.589%   -0.835%   -0.321%
  large_dataset  bench_query_population_property                               -0.798%   -1.328%   -0.294%

Unchanged:
  Group          Bench                                                          Change  CI Lower  CI Upper
  -------------  -----------------------------------------------------------  --------  --------  --------
  examples       example-births-deaths                                          0.114%   -0.427%    0.649%
  indexing       query_people_count_indexed_multi-property                      0.217%   -0.280%    0.785%
  indexing       query_people_multiple_individually_indexed_properties          0.816%   -0.008%    1.697%
  indexing       with_query_results_multiple_individually_indexed_properties   -0.097%   -1.179%    0.842%
  sample_people  sampling_single_l_reservoir                                    0.107%   -0.567%    0.850%
  large_dataset  bench_query_population_multi_unindexed                         0.006%   -0.347%    0.331%
  large_dataset  bench_query_population_multi_indexed                          -0.000%   -0.251%    0.336%

@cdc-as81 cdc-as81 merged commit 26dd313 into main Jan 8, 2026
3 checks passed
@cdc-as81 cdc-as81 deleted the cdc-as81-print-profiling branch January 8, 2026 22:35
@ext-cfa-ixa-release ext-cfa-ixa-release bot mentioned this pull request Jan 12, 2026
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.

Create a method on Context to print execution stats.

3 participants