global-electrification-platform / data-service

Data service of Global Electrification Platform
MIT License
2 stars 4 forks source link

Error ingesting range filters with timesteps #59

Closed vgeorge closed 5 years ago

vgeorge commented 5 years ago

After ingesting scenarios files, the CLI updates the model record with min/max values for range filters, but it not doing it properly for timesteped range filters. The query below attemps to compute min/max from a filter key that doesn't include a timestep on it and is not available. It should be changed to compute min/max values from all filter timesteps:

https://github.com/global-electrification-platform/data-service/blob/master/cli/models.js#L176-L189

vgeorge commented 5 years ago

Fixed with https://github.com/global-electrification-platform/data-service/pull/60.