josdejong/mathjs

Insufficient typing for `add`, `subtract`, `dotMultiply` etc

Open

#2.349 geöffnet am 9. Nov. 2021

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

Repository-Metriken

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

Beschreibung

The functions

lack sufficiently fine-grained type definitions. In particular, this produces an error:

math.add([1, 2], [3, 4])[0];
// Element implicitly has an 'any' type because expression
// of type '0' can't be used to index type 'MathType'.
// Property '0' does not exist on type 'MathType'

while it should be clear that the result is a one-dimensional array.

Adding some overloads would make the TypeScript experience with Mathjs much smoother.

Contributor Guide

Research-Richtung
Untersuche die Typdefinitionen in `types/index.d.ts` für die Funktionen `add`, `subtract`, `dotDivide`, `dotMultiply` und `dotPow`. Verstehe den aktuellen `MathType` Union Typ und wie Überladungen präzisere Rückgabetypen spezifizieren können. Sieh dir vorhandene Überladungen in derselben Datei als Leitfaden an. Füge Überladungen hinzu, die den Ergebnistyp korrekt ableiten, wenn die Eingaben Arrays sind.
Tech Stack
javascripttypescript
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
TypeScriptJavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75