Adding support for a built-in Pi value?

[jbranchaud]

I have started using bignumber.js for a project. I needed to do some math that involved the use of Pi in one of the equations. I had to construct a BigNumber for Pi myself and I wonder if it would make more sense to have this (and possibly other standard values) built-in?

Creating the Pi value wasn't straightforward either.

I tried BigNumber(Math.PI); but that gives an error because it has more than 15 significant digits.

This forced me to do something like BigNumber(Math.PI.toFixed(10));.

Seems like it would just be easier if we could do BigNumber.PI;.

[MikeMcl]

There will only be an error if a number type is passed, so BigNumber( Math.PI.toString() ) could be used.

It was tempting to include constants such as pi, but keeping the library lean (<= 5KB minified and gzipped) was a major goal.

I will shortly be releasing an extended version with a built-in parser which may include such.

Thanks for the feedback, and it will be further considered.

[josdejong]

+1 for including a bignumber versions of pi and e.