Code-Inspect / flowr

A program slicer and dataflow analyzer for the R programming language.
https://github.com/Code-Inspect/flowr/wiki
GNU General Public License v3.0
14 stars 2 forks source link

Include times per line in statistics #830

Closed Ellpeck closed 4 weeks ago

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 91.17647% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 71.92%. Comparing base (d347535) to head (44f7a64).

Files Patch % Lines
src/benchmark/summarizer/first-phase/process.ts 82.35% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #830 +/- ## ========================================== + Coverage 71.84% 71.92% +0.08% ========================================== Files 202 202 Lines 7274 7296 +22 Branches 1125 1227 +102 ========================================== + Hits 5226 5248 +22 - Misses 1691 1694 +3 + Partials 357 354 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Ellpeck commented 1 month ago

TODO: standard deviation for 1 run is 0 because it gets summarized from just one run, and then the standard deviation for the total is also 0 because it gets calculated based on the other standard deviations. that's wrong & bad

This and a lot of other problems will be solved by flipping summarization order (currently we summarize all runs of a file first, then summarize over those. we should be summarizing all files of the nth run, and then summarizing all runs)

EagleoutIce commented 4 weeks ago

This pull request is included in v2.0.3 (see Release v2.0.3 (Fine-Grained Benchmarks)).

EagleoutIce commented 4 weeks ago

This pull request is included in v2.0.3 (see Release v2.0.3 (Fine-Grained Benchmarks)).