smart-facility / petajakarta-web

Web files for PetaJakarta.org
http://petajakarta.org
Other
8 stars 6 forks source link

Duplicate times in floodgauge charts #70

Closed matthewberryman closed 8 years ago

matthewberryman commented 8 years ago

image

matthewberryman commented 8 years ago

db_and_logs.zip

tomasholderness commented 8 years ago

Pushed a fix to cognicity-reports-floodgauges, where timezone was set wrong and SQL rows loop was incorrect.

https://github.com/smart-facility/cognicity-reports-floodgauge/commit/b59b95077521f1e521fa4be8e4d88c52b1955776

matthewberryman commented 8 years ago

I have this running on dev. Will monitor (ignoring old data that I won't drop). You can bump cognicity-reports to have updated submodule if you like but I'm primarily focussing on the merge branch of cognicity-reports.

matthewberryman commented 8 years ago

Still not quite right. screenshot 2015-12-05 15 31 21

matthewberryman commented 8 years ago

@talltom looking at new data points coming in, it looks like you might be omitting the minutes when displaying?

tomasholderness commented 8 years ago

Check out latest pushes to floodgauge submodule. It was a problem with check for duplicate data.

https://github.com/smart-facility/cognicity-reports-floodgauge/commits/master

matthewberryman commented 8 years ago

Giving that a whirl now.

matthewberryman commented 8 years ago

Hard for me to check until either #72 is fixed or I could drop the tables and recreate if you like ?

matthewberryman commented 8 years ago

Seems to be resolving as new data comes in. Happy to close this off.