MachineLearningLifeScience / stochman

Algorithms for computations on random manifolds made easier
Apache License 2.0
86 stars 11 forks source link

Slight optimization by removing two BMM #27

Closed eugene closed 1 year ago

eugene commented 1 year ago

Results in ~3% speedup for the example code.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.06 :tada:

Comparison is base (122cdd8) 52.57% compared to head (a055a84) 52.63%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #27 +/- ## ========================================== + Coverage 52.57% 52.63% +0.06% ========================================== Files 9 9 Lines 1533 1535 +2 ========================================== + Hits 806 808 +2 Misses 727 727 ``` | Flag | Coverage Δ | | |---|---|---| | Linux | `52.63% <100.00%> (+0.06%)` | :arrow_up: | | Windows | `?` | | | cpu | `52.63% <100.00%> (+0.06%)` | :arrow_up: | | macOS | `52.63% <100.00%> (+0.06%)` | :arrow_up: | | pytest | `52.63% <100.00%> (+0.06%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MachineLearningLifeScience#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/MachineLearningLifeScience/stochman/pull/27?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MachineLearningLifeScience) | Coverage Δ | | |---|---|---| | [stochman/curves.py](https://codecov.io/gh/MachineLearningLifeScience/stochman/pull/27?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MachineLearningLifeScience#diff-c3RvY2htYW4vY3VydmVzLnB5) | `87.50% <100.00%> (+0.09%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MachineLearningLifeScience). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MachineLearningLifeScience)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.