yunruse / Noether

Just another units package
MIT License
9 stars 0 forks source link

`GeometricUnit` shouldn't be able to hold `LinearUnit` #59

Closed yunruse closed 1 year ago

yunruse commented 1 year ago

General info

In-development bug while trying to fix #54.

Steps

>>> byte / (minute & second)
byte / minute & second  # data_rate

Expected behaviour

A LinearUnit really shouldn't be stored in a GeometricUnit - it's very ill-defined. The smallest unit should be stored instead, i.e.

>>> byte / (minute & second)
byte / second  # data_rate