Skip to content

simplify unitless powers automatically#435

Merged
edzer merged 3 commits into
mainfrom
pow_unitless
Jun 23, 2026
Merged

simplify unitless powers automatically#435
edzer merged 3 commits into
mainfrom
pow_unitless

Conversation

@Enchufa2

Copy link
Copy Markdown
Member

Fixes #434, revisiting #179.

@edzer This is my proposal with no impact on other existing functionality. What do you think?

Comment thread R/arith.R
@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.45%. Comparing base (3854ba0) to head (34d9ece).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #435      +/-   ##
==========================================
+ Coverage   92.44%   92.45%   +0.01%     
==========================================
  Files          20       20              
  Lines        1138     1140       +2     
==========================================
+ Hits         1052     1054       +2     
  Misses         86       86              

☔ View full report in Codecov by Harness.
📢 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.

@jranke

jranke commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

LGTM, thanks for addressing this so quickly! Impressive!

@edzer

edzer commented Jun 23, 2026

Copy link
Copy Markdown
Member

Sure, LGTM!

@edzer
edzer merged commit 00250e2 into main Jun 23, 2026
30 checks passed
@Enchufa2
Enchufa2 deleted the pow_unitless branch June 23, 2026 23:10
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.

Use power operations with dimensionless units

3 participants