Skip to content

Conversation

@gvegayon
Copy link
Member

Improves data management. From using vector of pairs it now uses a single vector. Also reduces the number of hashing operations needed while adding stats. Next step is evaluating std::unordered_map::try_insert.

@codecov
Copy link

codecov bot commented Jan 24, 2022

Codecov Report

Merging #6 (6a4523a) into master (499d67b) will decrease coverage by 0.58%.
The diff coverage is 91.40%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   90.75%   90.16%   -0.59%     
==========================================
  Files          41       42       +1     
  Lines        2282     2339      +57     
==========================================
+ Hits         2071     2109      +38     
- Misses        211      230      +19     
Impacted Files Coverage Δ
include/barry/barray-bones.hpp 100.00% <ø> (ø)
include/barry/barray-meat.hpp 75.84% <ø> (-7.28%) ⬇️
include/barry/counters-meat.hpp 81.63% <ø> (ø)
include/barry/counters/phylo.hpp 99.30% <ø> (ø)
include/barry/model-bones.hpp 95.83% <ø> (ø)
...clude/barry/models/geese/geese-meat-likelihood.hpp 97.95% <ø> (ø)
include/barry/powerset-bones.hpp 100.00% <ø> (ø)
include/barry/rules-bones.hpp 100.00% <ø> (ø)
include/barry/support-bones.hpp 100.00% <ø> (ø)
include/barry/barraydense-meat.hpp 90.65% <80.95%> (+2.58%) ⬆️
... and 31 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 499d67b...6a4523a. Read the comment docs.

@gvegayon gvegayon merged commit bc23848 into master Jan 24, 2022
@gvegayon gvegayon deleted the statsdb branch January 24, 2022 20:55
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.

2 participants