wolph / alfred-converter

Alfred unit converter is a smart calculator for Alfred with support for unit conversions to make it a bit comparable to the Google Calculator and Wolfram Alpha.
93 stars 15 forks source link

Fix base units not being set #54

Closed aaronkollasch closed 1 year ago

aaronkollasch commented 1 year ago

For example millihertz was listed in units.base_units because its base_unit was set to None, and the unit wasn't picked up from <ConversionToBaseUnit>.

Also fixes the erroneous conversion 1 Hz = 6.283185 counts/second by giving hertz the base unit 6.283185 radians/second, the same as 1 c/s. (Note that mHz and all other scales of Hz are already based on radians/second.)

wolph commented 1 year ago

Nice fix, thank you!