EnergySystemsModellingLab / EnergySystemsModellingLab/MUSE2
Consider using `nonempty` crate
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Rust
- Sterne
- 8
- Forks
- 5
- Ø Merge
- 1 T. 2 Min.
- Gemergte PRs (30 T.)
- 37
Beschreibung
I've come across this crate and it seems like it could be useful: https://crates.io/crates/nonempty
It provides a `NonEmpty` type, which behaves (mostly) like a `Vec`, but guarantees that there is at least one element. Specifically for the input layer, there are a bunch of places where we want to ensure that `Vec`s are non-empty and this could be a nice way of expressing it...
What do you think @dalonsoa?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der Überprüfung der Eingabeschicht und der Stellen, an denen erwartet wird, dass Vec-Werte nicht leer sind. Lies die Dokumentation des nonempty-Crates, um zu beurteilen, ob sein NonEmpty-Typ für diese Verwendungen geeignet ist. Die Aufgabe ist abgeschlossen, wenn der Umfang der Übernahme festgelegt und die betroffene Eingabeverarbeitung konsistent aktualisiert wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100