search

ADI-Labs / density

wireless density API
13 stars 26 forks source link

Remove unnecessary connection param #211

Closed alexander-yu closed 7 years ago

alexander-yu commented 7 years ago

The predictor no longer takes in the connection to the database, so no need to pass the connection through; docstrings also updated to reflect this change. modified: density/data/__init__.py modified: density/density.py

alanhdu commented 7 years ago

LGTM, although I'm shocked flake8 didn't catch this before. What do you think about hooking up https://pypi.python.org/pypi/vulture to guard against this?

alexander-yu commented 7 years ago

Sounds good to me; I'll take a look into it.