-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
more error handling #26
Comments
Right. Here: https://github.com/IQSS/dataverse-metrics/blob/v0.2.5/download.py#L32 Thanks for the bug report. |
and these:
|
@donsizemore gotcha. Here: https://github.com/IQSS/dataverse-metrics/blob/v0.2.5/download.py#L94 Thanks for continuing to report these! Sorry for the trouble! Oh, over in #27 I'm suggesting dropping support for Python 2. Of course, this is not at all a requirement for improving the error handling. It's just tricky with all the requirements I've imposed on myself. In https://github.com/IQSS/dataverse-sample-data I'm experimenting with different requirements: Python 3 only, pip install of dependencies. A different and probably more normal way of doing things instead of (old) "batteries included". |
one more error code (i'm not concerned about these, just dutifully reporting them). on python2/3 — any OS should be able to easily run 3.
|
have I exhausted my repertoire? I think not:
|
i've tasted blood, and i want more (more! more! more!)
|
would be great to handle these:
The text was updated successfully, but these errors were encountered: