openreview-py

openreview-py copied to clipboard

allow openreview.Client to run as a context manager

1

e.g. ``` with openreview.Client() as client: notes = client.get_notes( ... ) ``` https://jeffknupp.com/blog/2016/03/07/python-with-context-managers/

mspector

openreview-py should support python 3

1

Importing the module in a python 3 environment throws the following error: ``` >>> import openreview Traceback (most recent call last): File "", line 1, in File "/usr/local/lib/python3.6/site-packages/openreview/__init__.py", line 2,...

zbialecki

mspector

mspector

for Northeastern project

mspector

source: https://stackoverflow.com/questions/16511337/correct-way-to-try-except-using-python-requests-module ``` url='http://www.google.com/blahblah' try: r = requests.get(url,timeout=3) r.raise_for_status() except requests.exceptions.HTTPError as errh: print ("Http Error:",errh) except requests.exceptions.ConnectionError as errc: print ("Error Connecting:",errc) except requests.exceptions.Timeout as errt: print ("Timeout Error:",errt)...

melisabok

Usually we get this error: ``` File "//anaconda/lib/python2.7/json/decoder.py", line 382, in raw_decode raise ValueError("No JSON object could be decoded”) ``` catch this value and return an exception. Also we can...

melisabok

error messages are very uninformative

3

need to refactor the error handling so that the proper error messages are displayed

mspector

example: client.get_groups(id="auai.org/UAI/2017/Paper.*/AnonReviewer.*") should return all groups that match the wildcard pattern.

mspector

note and tag don't have the field 'active' anymore

1

Please remove it from the library

melisabok