replace logging.warning with warnings.warn, remove logging.debug

[wholmgren]

Will anyone miss these internal pvlib logging calls?

With these changes, grep logg pvlib/* -R returns nothing.

• Closes logging.basicConfig in __init__.py causes misbehavior after import #447
• Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests must pass on the TravisCI and Appveyor testing services.
• Code quality and style is sufficient. Passes git diff upstream/master -u -- "*.py" | flake8 --diff and/or landscape.io linting service.
• New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
• Updates entries to docs/sphinx/source/api.rst for API changes.
• Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.

[mikofski]

No I won't miss them. Less to maintain if we cut them out. My vote is to approve this PR.

[cwhanse]

I agree, we can remove the logging calls

…

Sent from my iPhone On Apr 6, 2018, at 12:38 PM, Mark Mikofski <[email protected]<mailto:[email protected]>> wrote: No I won't miss them. Less to maintain if we cut them out. My vote is to approve this PR. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#448 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFJNL_kKa5w2IupCi_QvjOi5bpoHe1jBks5tl7YhgaJpZM4TKXr6>.

[markcampanelli]

Seems reasonable.