The modified source code of METIS has got in, followed by its Cython wrappers. Thanks again to Yingchong Situ for all his legacy work. Nevertheless, things were not smooth and there were lots of hiccups and things to learn.
One of the modules in the package was named types which was being imported by absolute import. Unknown of the fact that types is also a built-in module of python, the situation was a mystery for me. Thanks to iPython which told me this
In [2]: types
Out[2]: <module 'types' from '/usr/lib/python2.7/types.pyc'>
This alerted me for all the difference, pros and cons of absolute and relative import. Now one may ask (Does anyone read these blog posts?) why didn't I go with the following at the very first place.
In [3]: from . import types
Actually networkx-metis is supposed to be installed as a namespace package in networkx, and the presence of __init__.py is prohibited in a namespace package. Hence from . import types would raise a Relative import from a non-package error.
We are now following the Google's style guide for Python[1].
Being licensed under Apache License Version 2, we also had to issue a file named NOTICE clearly stating the modifications we did to the library networkx-metis is a derivative work of.
Next important items in my TODO list are
- Finalizing everything according for namespace packaging
- Setting up Travis CI
- Hosting docs over readthedocs.org
That's all for now.
Happy Coding!
[1] https://google-styleguide.googlecode.com/svn/trunk/pyguide.html
No comments:
Post a Comment