ADI-Labs / density

wireless density API
13 stars 26 forks source link

Offset issues #99

Closed maclyn closed 9 years ago

maclyn commented 9 years ago

I was trying to write the documentation for Density and ran a few test queries, and I got the "admin was notified error". I think my query, which was http://density.adicu.com/window/2014-10-10T08:00/2014-10-10T21:30/building/75?auth_token=[real auth token], was in the right format (if not, we should return an error for invalid parameter(s).

The error was:

Traceback (most recent call last): File "/srv/adi-density/www/local/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request rv = self.dispatch_request() File "/srv/adi-density/www/local/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request return self.view_functionsrule.endpoint File "/srv/adi-density/www/density/density.py", line 90, in authorization_checker return func(_args, *_kwargs) File "/srv/adi-density/www/density/density.py", line 300, in get_window_building_data new_offset = offset + db.QUERY_LIMIT TypeError: unsupported operand type(s) for +: 'NoneType' and 'int

thebrianzeng commented 9 years ago

yeah i got the errors - try it again now, it should be fixed

On Fri Jan 30 2015 at 11:29:36 PM Maclyn B notifications@github.com wrote:

I was trying to write the documentation for Density and ran a few test queries, and I got the "admin was notified error". I think my query, which was http://density.adicu.com/window/2014-10-10T08:00/2014-10-10T21:30/building/75?auth_token=KXTNRB6JBVQYOPFALF04W6Y9ZILMLBTS, was in the right format (if not, we should return an error for invalid parameter(s).

The error was:

Traceback (most recent call last): File "/srv/adi-density/www/local/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request rv = self.dispatch_request() File "/srv/adi-density/www/local/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request return self.view_functionsrule.endpoint http://**req.view_args File "/srv/adi-density/www/density/density.py", line 90, in authorization_checker return func(_args, *_kwargs) File "/srv/adi-density/www/density/density.py", line 300, in get_window_building_data new_offset = offset + db.QUERY_LIMIT TypeError: unsupported operand type(s) for +: 'NoneType' and 'int

— Reply to this email directly or view it on GitHub https://github.com/adicu/density/issues/99.

maclyn commented 9 years ago

Great, it seems to work again. I was getting errors for the day route as well; I'm not sure if that's related...

maclyn commented 9 years ago

I think we can close this now. It seems to be fixed. :closed_book: