[GRASS-dev] [GRASS GIS] #3010: PyGRASS fails to write vector map with attributes

GRASS GIS trac at osgeo.org
Fri Nov 11 05:18:10 PST 2016


#3010: PyGRASS fails to write vector map with attributes
-------------------------+-------------------------------------------------
  Reporter:  annakrat    |      Owner:  grass-dev@…
      Type:  defect      |     Status:  reopened
  Priority:  major       |  Milestone:  7.2.0
 Component:  PyGRASS     |    Version:  svn-trunk
Resolution:              |   Keywords:  vector, API break, backwards
       CPU:              |  compatibility
  Unspecified            |   Platform:  All
-------------------------+-------------------------------------------------

Comment (by wenzeslaus):

 Replying to [comment:5 huhabla]:
 > IMHO the former implementation was close to be a bug that was fixed in
 trunk.

 If it was buggy, than there should be a special `if` with an error message
 (exception, fatal, assert) which will tell you that you are using old API
 which was buggy and you need to change your code.

 BTW, the current message is not that good even in context of the 7.2 API
 only because you don't know what has the wrong type.

 > For every new feature a new category was created by default, except a
 category was already attached to the c_cat field in the geometry.

 This does not seem buggy but as a simple one which not flexible enough for
 advanced user but pretty good for new users.

 > It was not obvious howto use the same category for different features.

 This sounds like a problem with documentation or a need for additional
 API, seems like a long jump to breaking backwards compatibility.

 Don't take me wrong, I agree that the new API is needed. It just needs
 some better handling for the 7.0 API. (We are getting complains about
 breaking API for modules between 6.4 and 7.0, so I suppose users care
 about backwards compatibility.)

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3010#comment:7>
GRASS GIS <https://grass.osgeo.org>



More information about the grass-dev mailing list