Skip to content

Conversation

araujoms
Copy link
Collaborator

It turns out the same technique applies to the power mean cone, allowing a rather simple solution.

Copy link

codecov bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.76%. Comparing base (a6c2fc2) to head (3402d4e).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #860      +/-   ##
==========================================
+ Coverage   96.72%   96.76%   +0.03%     
==========================================
  Files          56       56              
  Lines        9102     9106       +4     
==========================================
+ Hits         8804     8811       +7     
+ Misses        298      295       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@araujoms
Copy link
Collaborator Author

Because it gets caught by

if all(isequal(inv(T(d))), cone.α)
and thus uses the code from hypogeomean. I've removed the unused branch.

@araujoms araujoms merged commit 7bec3f3 into master Sep 29, 2025
8 checks passed
@araujoms araujoms deleted the araujoms/centralpowermean branch September 29, 2025 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants