AcademySoftwareFoundation / rawtoaces

RAW to ACES Utility
141 stars 47 forks source link

"spectral_data.index" attribute value should be fully specified in "data/training/training_spectral.json". #112

Closed KelSolaar closed 11 months ago

KelSolaar commented 5 years ago

Hi,

The spectral_data.index attribute value in the data/training/training_spectral.json file uses an ellipsis which is bad from a parsing standpoint:

https://github.com/ampas/rawtoaces/blob/25856b410497c57acb4f9e25f7840a0621c3a318/data/training/training_spectral.json#L22

While it is convenient from an authoring perspective it makes parsing unnecessarily complicated if one tries to correlate the indexes with the spectral data columns: One need to parse and interpolate the numbering in the existing indexes strings to build the remaining ones.

Cheers,

Thomas