josdejong/mathjs

UnaryMinus around ConstantNode not mached by "c" in simplify

Open

#1.406 geöffnet am 15. Feb. 2019

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.298 Forks)batch import
featurehelp wanted

Repository-Metriken

Stars
 (13.832 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die simplify Funktion im mathjs Quellcode, um zu verstehen, wie unaryMinus Knoten abgeglichen werden. Erwägen Sie das Hinzufügen einer Option, um unaryMinus von Konstanten als Konstanten zu behandeln.
Tech Stack
javascript
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75