josdejong/mathjs

Avoid unnecessary parentheses for Units with complex value when using `format`

Open

#3,614 创建于 2025年12月17日

在 GitHub 查看
 (4 评论) (0 反应) (0 负责人)JavaScript (13,832 star) (1,298 fork)batch import
category:unitsfeaturehelp wanted

描述

Describe the bug Some units are considered complex units, meaning they are treated differently to normal, "numeric" units. This for example is the case for the volt-ampere reactive, considered to be VAR by mathjs.

To Reproduce I am unsure why, but converting these units leads to unexpected results:

(new Unit(1, 'MW')).to('kW').format() // 1000 kW - correct
(new Unit(1, 'MVAR')).to('kVAR').format() // (1000) kVAR - weird
(new Unit(1, 'MVAR')).to('kW').format() // (1000i) kW - weird

I understand the want to represent these as complex numbers, but I think this is fairly out of place when dealing with units. It leads to unexpected conversions and format outputs.

Notice that using formatUnits avoids this, of course, since only the number part is affected by this:

(new Unit(1, 'MVAR')).to('kVAR').formatUnits() // kVAR
(new Unit(1, 'MVAR')).to('kW').formatUnits() // kW

贡献者指南