search

developmentseed / tecnico-energy-app

https://dashboard-ds-peach.vercel.app/
0 stars 0 forks source link

Use pre-aggregated max for tile route #46

Closed Tammo-Feldmann closed 3 months ago

Tammo-Feldmann commented 3 months ago

What I'm changing

I'm making use of the pre-aggregated scenario_metrics_table in the tile route to save on extra computation in the query.

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tecnico-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2024 11:37pm
Tammo-Feldmann commented 3 months ago

Looks good, however I think this could be simpler in that we're no longer using the scenario_metrics table. Rather than adding a metrics_total_table arg, we can just use "scenario_metrics_total" in place of metrics_table

Dang, you pay close attention @alukach. I should have made that change myself. Thank you for reading so closely. I'll keep this in mind for next time.