search

ADI-Labs / density

wireless density API
13 stars 26 forks source link

Removed midnight of next day from /day routes #85

Closed thebrianzeng closed 9 years ago

thebrianzeng commented 9 years ago

/day routes were returning the first result for the next day, which we do not want.

natebrennand commented 9 years ago

wait, the SQL queries should be rewritten to use < rather than <=

thebrianzeng commented 9 years ago

we could do that too, but that would also change the /window functionality

On Fri Jan 30 2015 at 4:04:45 PM Nate notifications@github.com wrote:

wait, the SQL queries should be rewritten to use < rather than <=

— Reply to this email directly or view it on GitHub https://github.com/adicu/density/pull/85#issuecomment-72270040.

natebrennand commented 9 years ago

but doesn't that change it properly to what it should be? otherwise when someone queried on multiple window ranges they wouldn't receive any data points multiple times

thebrianzeng commented 9 years ago

ahh good point, will change

On Fri Jan 30 2015 at 4:24:12 PM Nate notifications@github.com wrote:

but doesn't that change it properly to what it should be? otherwise when someone queried on multiple window ranges they wouldn't receive any data points multiple times

— Reply to this email directly or view it on GitHub https://github.com/adicu/density/pull/85#issuecomment-72272699.