Python 3 compatiability

[datapolitan]

Any chance this could work with Python 3? I have a student in class who can't use this because he isn't using Python 2.

pip install nyc-geoclient fails on the print statement in setup.py line 12. Not sure what else would go into making this work in Python 3.

[nygeog]

Could maybe have the student change that print code... print will be a function in python 3.

source:
http://stackoverflow.com/questions/826948/syntax-error-on-print-with-python-3

[fractalphile]

For what it's worth, in case anyone else is trying this, I found three things that needed changing to work with Python 3.4:

1. The print statement in setup.py Line 15 needs parentheses.
2. In api.py Line 9, import urlencode from urllib.parse instead of just urllib.
3. Because iterating over dictionaries works differently in python 3 (see here), lines 53-55 don't work. I changed the original code

        # Ensure no 'None' values are sent to server
        for k in kwargs.keys():
            if kwargs[k] is None:
                kwargs.pop(k)

to:

        # Ensure no 'None' values are sent to server
        listOfNone = []
        for k in kwargs.keys():
            if kwargs[k] is None:
                listOfNone.append(k)
        for n in listOfNone:
            kwargs.pop(n)

and that seemed to work. With the caveat that I'm pretty inexperienced, so no guarantees!

[ResidentMario]

For future reference, if you need it I suggest using my fork.

[ishiland]

I incorporated some of the changes in previous PR's and added some tests.

If there is enough interest, I would be glad to put my fork on pypi and actively maintain it. I'm just not sure what an appropriate namespace for it is.

[ResidentMario]

I think that would be apropro, as this version of the library has been abandoned for quite a long time now. 😄