Astropy: Move generic xml methods outside io.votable?

Created on 5 Mar 2018  ·  3Comments  ·  Source: astropy/astropy

In PyVO, i am using the astropy xml ecosystem to parse the VOSI and UWS interfaces.

There are several methods and classes in io.votable which i re-use for those interfaces and thus this generic functionallity would possibly be better placed (and renamed) in utils.xml

This includes (subject to change):

  • eveything in io.votable.exceptions except the W and E prefixed classes.
  • nearly everything in io.votable.xmlutil, altough there are some votable-specific warnings raised here, so a minor rewrite would be necessary.
  • convert_to_writable_filelike from io.votable.util

Without deeper analysis i cannot say if there would be more to change; does this qualifies for an APE?

Feature Request Refactoring io.votable

Most helpful comment

Are you looking for a maintainer?
Would raise my hand, as i already know the code and the VO.

All 3 comments

Currently, io.votable has no lead maintainer. As a deputy, I am not comfortable in making such non-trivial refactoring decision. Therefore, I would like to defer to @astropy/coordinators .

Are you looking for a maintainer?
Would raise my hand, as i already know the code and the VO.

@funbaker , that is the decision for @astropy/coordinators but FWIW :+1: from me!

Was this page helpful?
0 / 5 - 0 ratings