Skip to content

Conversation

@WizardOfMenlo
Copy link
Owner

So after some hiatus, I am back. This pull request imports the group that Ruth gracefully provided, and adds some machinery in benchmark to test them as well! We will most likely want to change a bit which sets are benchmarked, but this is most of what we want.

@codecov
Copy link

codecov bot commented Oct 29, 2020

Codecov Report

Merging #83 (10b9604) into master (089985d) will decrease coverage by 0.62%.
The diff coverage is 100.00%.

❗ Current head 10b9604 differs from pull request most recent head a926ff8. Consider uploading reports for the commit a926ff8 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
- Coverage   95.26%   94.64%   -0.63%     
==========================================
  Files          43       39       -4     
  Lines        3488     3157     -331     
==========================================
- Hits         3323     2988     -335     
- Misses        165      169       +4     
Impacted Files Coverage Δ
src/perm/impls/standard.rs 96.42% <100.00%> (ø)
src/perm/impls/sync.rs 94.11% <100.00%> (ø)
src/group/orbit/abstraction.rs 89.47% <0.00%> (-10.53%) ⬇️
src/group/random_perm.rs 95.77% <0.00%> (-4.23%) ⬇️
src/perm/utils.rs 96.61% <0.00%> (-3.39%) ⬇️
src/group/stabchain/mod.rs 94.78% <0.00%> (-3.15%) ⬇️
src/perm/actions.rs 97.67% <0.00%> (-2.33%) ⬇️
src/group/utils.rs 97.75% <0.00%> (-2.25%) ⬇️
src/group/stabchain/element_testing.rs 97.41% <0.00%> (-1.94%) ⬇️
src/group/stabchain/builder/random/random_ift.rs 91.59% <0.00%> (-1.92%) ⬇️
... and 20 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 089985d...a926ff8. Read the comment docs.

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