josdejong/mathjs

Functions do not recon with `config.number` when converting a string into a numeric type

Open

#3,206 opened on 2024年5月29日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)JavaScript (13,832 stars) (1,298 forks)batch import
bughelp wanted

説明

It's a bit of an edge case, but for example:

math.config({number: 'BigNumber'})
math.add('2', '3')
// results in a number with value 5, but should be a BigNumber with value 5

The reason is that functions like add do not have a signature like add(string, string). Therefore, automatic conversion kicks in. The automatic conversion finds a conversion from string -> number that fits the bill and uses that, without reconing with the configured config.number. I'm not sure how a proper solution to fix this would look like, since the type conversions are static 🤔.

コントリビューターガイド

技術スタック
javascript
領域
backend
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Knowledge of mathjs configurationUnderstanding of function signatures
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
45
調査方針
Investigate how mathjs functions handle argument type conversions, specifically in the `add` function. Look at the `conversion` module to understand how conversions are prioritized. Consider modifying the automatic conversion logic to respect `config.number` when converting strings to numeric types. The issue mentions that conversions are static, so a solution might involve dynamic conversion selection based on configuration.