Skip to content

Add lb/mil² and Wh/lbs #1092

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 30, 2022
Merged

Add lb/mil² and Wh/lbs #1092

merged 2 commits into from
May 30, 2022

Conversation

rutemja
Copy link
Contributor

@rutemja rutemja commented May 30, 2022

Fixes #1091 (PoundForcePerSquareMil
Fixes #1090 (WattHourPerPound)

@codecov
Copy link

codecov bot commented May 30, 2022

Codecov Report

Merging #1092 (af51458) into master (8afabd2) will increase coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #1092   +/-   ##
======================================
  Coverage      85%     85%           
======================================
  Files         315     315           
  Lines       48697   48751   +54     
======================================
+ Hits        41564   41618   +54     
  Misses       7133    7133           
Impacted Files Coverage Δ
...ions/GeneratedCode/NumberToPressureExtensions.g.cs 100% <100%> (ø)
...eneratedCode/NumberToSpecificEnergyExtensions.g.cs 100% <100%> (ø)
UnitsNet/GeneratedCode/Quantities/Pressure.g.cs 92% <100%> (+<1%) ⬆️
...tsNet/GeneratedCode/Quantities/SpecificEnergy.g.cs 89% <100%> (+<1%) ⬆️

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 8afabd2...af51458. Read the comment docs.

Copy link
Owner

@angularsen angularsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks perfect to me, great work!

@@ -51,6 +53,8 @@ public class PressureTests : PressureTestsBase

protected override double PoundsForcePerSquareInchInOnePascal => 1.450377377302092e-4;

protected override double PoundsForcePerSquareMilInOnePascal => 1.450377377302092e-10;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could not find a converter source to verify, but since mil is 1e-3 inch and square inch is 1e-6, then this seems reasonable.

@@ -22,6 +22,11 @@ public class SpecificEnergyTests : SpecificEnergyTestsBase
protected override double MegawattHoursPerKilogramInOneJoulePerKilogram => 2.77777778e-10;
protected override double GigawattHoursPerKilogramInOneJoulePerKilogram => 2.77777778e-13;

protected override double WattHoursPerPoundInOneJoulePerKilogram => 1.2599788055556e-4;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@angularsen
Copy link
Owner

Nuget should be out in a few minutes.

Release UnitsNet/4.132.0 · angularsen/UnitsNet

@rutemja
Copy link
Contributor Author

rutemja commented May 31, 2022

Thanks :)

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.

Add FromPoundsForcePerSquareMil (lbs/mil²) to Pressure Add KilowattHourPerPound (kWh/lbs) to SpecificEnergy
2 participants