claircore icon indicating copy to clipboard operation
claircore copied to clipboard

libindex: add some guess at layer preallocation sizing

Open hdonnay opened this issue 1 year ago • 2 comments

This is an idea I had that I threw together.

hdonnay avatar Jan 10 '24 15:01 hdonnay

Codecov Report

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

Project coverage is 56.45%. Comparing base (417c2b0) to head (134bdde). Report is 4 commits behind head on main.

:exclamation: Current head 134bdde differs from pull request most recent head 5670344. Consider uploading reports for the commit 5670344 to get more accurate results

Files Patch % Lines
libindex/fetcher.go 89.47% 1 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1202      +/-   ##
==========================================
+ Coverage   55.33%   56.45%   +1.11%     
==========================================
  Files         265      233      -32     
  Lines       16574    15370    -1204     
==========================================
- Hits         9172     8677     -495     
+ Misses       6451     5807     -644     
+ Partials      951      886      -65     

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

codecov[bot] avatar Jan 10 '24 18:01 codecov[bot]

This uses the mean, but probably shouldn't. Something like a running median would be better, but would require an indexed skip list implementation.

hdonnay avatar Mar 25 '24 19:03 hdonnay