{"issue":{"state":"open","pull_request":{"diff_url":"https://github.com/astropy/astropy/pull/46.diff","html_url":"https://github.com/astropy/astropy/pull/46","patch_url":"https://github.com/astropy/astropy/pull/46.patch"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/8988285911e50fbe248b659d43bccf83?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"8988285911e50fbe248b659d43bccf83","url":"https://api.github.com/users/mdboom","login":"mdboom","id":38294},"milestone":null,"assignee":null,"updated_at":"2011-11-08T13:21:02Z","labels":[],"closed_at":null,"comments":20,"url":"https://api.github.com/repos/astropy/astropy/issues/46","body":"This is an initial import of vo.table into astropy as astropy.io.vo.\r\n\r\nThis includes only the VOTABLE file format handling code.  The VO networking (conesearch, ssa, image search etc.) stuff is still kind of half-baked and not ready -- that will live on a branch on my fork for a while.\r\n\r\nThere are some things left to do, but they are perhaps best left for after merging:\r\n\r\n  - There are lots of general-purpose utilities here that should be moved into astropy.utils.\r\n\r\n  - There are some local data files which should use the new data loading framework when that lands on master.\r\n\r\n  - It has an optional dependency on \"xmllint\" for schema verification.  We should add this to the forthcoming \"check for optional dependencies\" tool.  At the moment, an exception is raised when calling the single function that depends upon \"xmllint\".  The tests will pass without \"xmllint\" installed, but the output of the tests will not be verified against the schema.\r\n\r\nSome points:\r\n\r\n  - This includes all of expat verbatim in cextern.  It does not build with a system copy of expat because at least on Ubuntu and Fedora, the system expat is built with UTF-8 strings rather than UCS2/4 strings which makes the conversion to Python Unicode strings much slower.  In benchmarking I found this to  have a factor of about 15% speedup.  The Python expat wrappers perform even worse because they do a lot of unnecessary work.  So, somehow, I suppose it should be documented that even though we are including expat in cextern, we don't want to create the option for the user to build against a system expat.\r\n","html_url":"https://github.com/astropy/astropy/issues/46","title":"VO","number":46,"id":2134493,"created_at":"2011-11-03T16:42:32Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/8988285911e50fbe248b659d43bccf83?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"8988285911e50fbe248b659d43bccf83","url":"https://api.github.com/users/mdboom","login":"mdboom","id":38294},"updated_at":"2011-11-08T13:21:02Z","url":"https://api.github.com/repos/astropy/astropy/issues/comments/2667295","body":"@eteq: I think this latest set of commits addresses everything you raised.","id":2667295,"created_at":"2011-11-08T13:21:02Z"}}