josdejong/mathjs

UnaryMinus around ConstantNode not mached by "c" in simplify

Aperta

#1406 aperta il 15 feb 2019

Vedi su GitHubGuida contributor
 (3 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

A unaryMinus wrapped around a ConstantNode is matched by "v", not "c", in the simplify function. While this matches the documentation, it is not what is expected for a "constant" and a "non-constant". Other single variable functions with a constant node are simplified to their result, and thus can be matched with "c". If existing compatibility presents an issue, this could be enabled as an option of the simplify function that defaults to the old behavior or by adding two new simplification selectors.

Guida contributor

Direzione di ricerca
Esaminare la funzione simplify nel codice sorgente di mathjs per capire come vengono abbinati i nodi unaryMinus. Considerare l'aggiunta di un'opzione per trattare unaryMinus di costanti come costanti.
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75