You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As per my comments from #1200 here are the proposed modifications to the
operators:
- [x] adding an `Inverse` relation between `Density` and
`SpecificVolume` (replacing the existing multiplication)
- [x] adding standard multiplication for `MolarEnergy.JoulePerMole *
AmountOfSubstance.Mole` (returning `Energy.Joule`)
- [x] adding standard multiplication for `Illuminance.Lux *
Area.SquareMeter` (returning `LuminousFlux.Lumen`)
- [x] `TemperatureDelta.Kelvin *
CoefficientOfThermalExpansion.PerKelvin` now returns a `Ratio` (instead
of a `double`)
- [x] added the `-- NoInferredDivision` to the
`KinematicViscosity.SquareMeterPerSecond * Duration.Second` operator (as
the opposite of the relation was `Area / Duration` ->
`KinematicViscosity` 😏 )
- [x] added the `-- NoInferredDivision` to the `Area.SquareMeter *
ReciprocalArea.InverseSquareMeter` operator (as the opposite of the
relation was `Ratio / Area` -> `InverseArea` does not appear intuitive )
- [x] change the multiplication units (using the `BaseUnit`) for the
`Mass.Kilogram = AmountOfSubstance.Mole * MolarMass.KilogramPerMole`
- [x] change the multiplication units (using the `BaseUnit`) for the
`AmountOfSubstance.Mole = MolarFlow.MolePerSecond * Duration.Second`
I've included tests for **_all_** operators from the affected classes
(_most operators, introduced with the `UnitRelations.json` were never
covered by tests_).
PS There are still many operators not covered by tests, but I'd rather
finish them later (_hopefully once we no longer have to worry about
rounding errors_).
---------
Co-authored-by: Andreas Gullberg Larsen <[email protected]>
0 commit comments