[GRASS-dev] [GRASS GIS] #2505: v.build.polylines options aren't working

[GRASS GIS]

#2505: v.build.polylines options aren't working
--------------------------+---------------------------------
  Reporter:  hellik       |      Owner:  grass-dev@…
      Type:  defect       |     Status:  new
  Priority:  major        |  Milestone:  6.4.5
 Component:  Vector       |    Version:  svn-releasebranch64
Resolution:               |   Keywords:  v.build.polylines
       CPU:  Unspecified  |   Platform:  All
--------------------------+---------------------------------

Comment (by hellik):

 Replying to [comment:11 mlennert]:
 > Replying to [comment:10 hellik]:
 > > Replying to [comment:7 mlennert]:
 > > > Replying to [comment:6 neteler]:
 > > > > Replying to [comment:2 neteler]:
 > > > > > Replying to [comment:1 mmetz]:
 > > > > > > Replying to [ticket:2505 hellik]:
 > > > > > >
 > > > > > > `cats=no` was indeed not working, fixed in r63321,r63322
 (trunk, relbr70).
 > > > > >
 > > > > > Should this go also into GRASS 6?
 > > > >
 > > > > Still unclear.
 > > >
 > > > What is actually unclear for me is what the problem was.
 > > >
 > >
 > > as mentioned above:
 > >
 > > ''cats=no was indeed not working, fixed in r63321,2 (trunk,
 relbr70).''
 > >
 >
 > I understand what the declared issue was and saw the fix, but what is
 weird is that using current trunk, when I undo the fix by commenting out
 the 'Vect_reset_cats(Cats);' call that was added, v.build.polyline still
 works as expected with your test data. And everything works as expected
 for me in grass64release as well, without the fix.
 >
 > So I'm not sure that the "fix" actually solved the problem.
 >
 > Moritz


 a few months ago some issues with v.build.polylines were reported in the
 MLs.

 at that time I've tested a lot, ''cats=no'' didn't work in any case.

 as my C knowledge is about zero, I can't judge if the fix mentioned in the
 ticket or any other commit 'fixed' this; but it is fixed now which was
 tested in the last days.

 taken from the ML at that time:

 {{{
 >But I’ve tried using cat=first and it continues to assign multiple cats
 to
 the segments.
 }}}

 there may be hidden some more issues...

 in quick summary in order to get solution to this ticket:

 - cats=first, cats=multi, cats=no: tested here with g7 and trunk => works
 now as expected

 - no filtering database entries (see above) => manual in g7 and trunk was
 expanded

 - v.build.polylines seems to work as expected in the g6-series

 as all seems to be fixed or working as normal, I suggest to close the
 ticket. feel free to reopen the ticket.


 Helmut

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2505#comment:12>
GRASS GIS <http://grass.osgeo.org>