josdejong/mathjs

math.rationalize() made some mistakes

Aperta

#2080 aperta il 12 gen 2021

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)JavaScript (1314 fork)batch import
featurehelp wanted

Metriche repository

Star
 (15.065 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

So I'm extremely happy with math.js, but I've had some problems with the following. Does anyone have a fix, I'm not sure what to do.

(a - b) + (a + 2b) was solved as a - b + a + 2 b and it should be 2a + b

2y + (3 - 4x) * 2 - 7x was solved as 2 y - 8 * x - 7 * x + 6 and it should be -15x + 2y + 6

2y + (3 + 4x) * 3 - 8x was solved as 2 y - 8 * x + 9 + 12 * x and it should be 4x + 2y + 9

Guida contributor

Direzione di ricerca
Esamina l'implementazione della funzione rationalize e scrivi test case per riprodurre le semplificazioni errate. Controlla come vengono gestiti la precedenza degli operatori e le parentesi.
Tech stack
javascript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptBasic algebra
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65