BlueQuartzSoftware / BlueQuartzSoftware/SIMPL
DataArrayProxy converting dimensions from double to size_t from JSON
- Lingua principale
- C++
- Stelle
- 8
- Fork
- 22
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia individuando DataArrayProxy::readVector e seguendo i suoi chiamanti per le dimensioni di tuple e component. Esamina i controlli di intervallo JSON esistenti e le aspettative di compatibilità dei file di pipeline; il lavoro è completato quando le dimensioni eccessive o non valide falliscono con una spiegazione senza compromettere i file esistenti supportati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- data
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100