BlueQuartzSoftware / BlueQuartzSoftware/SIMPL
DataArrayProxy converting dimensions from double to size_t from JSON
- Vorherrschende Sprache
- C++
- Sterne
- 8
- Forks
- 22
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
In `DataArrayProxy::readVector`, an array of numbers is read from JSON and converted to a `std::vector`. This function is used to read the tuple and component dimensions from JSON. This function does some checks on the range of acceptable values; however, it does not report any errors and has an implicit conversion between `double` and `size_t` inside its check. For exceedingly large datasets this would be an issue when attempting to read and write their dimensions.
A possible solution would be to store the dimensions as a string though this would affect backwards compatibility in pipeline files. Additionally better error reporting should be implemented to tell the user why the read failed.
Beitragsleitfaden
Rechercherichtung
Beginne damit, DataArrayProxy::readVector zu finden und seine Aufrufer für Tuple- und Component-Dimensionen nachzuverfolgen. Prüfe die bestehenden JSON-Bereichsprüfungen und die Kompatibilitätserwartungen für Pipeline-Dateien; abgeschlossen ist die Aufgabe, wenn übergroße oder ungültige Dimensionen mit einer Erklärung fehlschlagen, ohne die Unterstützung vorhandener Dateien zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- data
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100