LinalgScalar does not have to imply Sub and Div

Offen

#737 erstellt am 02.10.2019 10 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 48/100

Issue-Typ: Refactoring
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: rust
Bereich: backend

Rechercherichtung

Start with src/linalg_traits.rs at the linked LinalgScalar definition, then inspect its uses and run the relevant test suite to identify which bounds are required. Done means the unnecessary Sub, Div, Add, and Mul bounds are relaxed without breaking compilation or existing tests.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Recall the definition of LinalgScalar:
https://github.com/rust-ndarray/ndarray/blob/eb82c93f0147df38e061597221ece3627f119a60/src/linalg_traits.rs#L18-L28

However, no algorithm involving LinalgScalar requires the scalar to be invertible in the additive group and multiplicative semigroup, in this crate at the very least. Therefore, Sub and Div trait bounds are not used. It would be more flexible if Sub and Div trait bounds are relaxed.

In addition, Add<Self, Output=Self> and Mul<Self, Output=Self> is already implied by num::Zero and num::One, respectively. These trait bounds can be relaxed, too.

Vorherrschende Sprache: Rust
Sterne: 4.3k
Forks: 391
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus rust-ndarray/ndarray

.is_all_nan() returns true for an all-finite array Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100

rust-ndarray/ndarray#1612 · 1 Kommentar · Erstellt am 08.8.2026
failed Send inference with ndarray v0.17 and uom 0.38 Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

rust-ndarray/ndarray#1617 · 1 Kommentar · Erstellt am 09.9.2026
Stack overflow in `triu` Offen

bug good first issue

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

rust-ndarray/ndarray#1615 · 1 Kommentar · Erstellt am 08.9.2026
Type covariance absent when used in struct Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

rust-ndarray/ndarray#1610 · Erstellt am 27.7.2026
Empty Array lead to "The strides must not allow any element to be referenced by two different indices" Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100

rust-ndarray/ndarray#1609 · Erstellt am 23.7.2026

Alle Issues in rust-ndarray/ndarray