[GRASS-dev] [GRASS GIS] #1016: v.info support for level 1 (patch)

#1016: v.info support for level 1 (patch)
-------------------------+--------------------------------------------------
Reporter: neteler | Owner: grass-dev@lists.osgeo.org
     Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.1
Component: Vector | Version: svn-develbranch6
Keywords: | Platform: All
      Cpu: All |
-------------------------+--------------------------------------------------
Attached the backport of support for level 1 by Markus Metz in v.info for
6.4.1.
This is needed to make r.fillnulls potentially faster/less memory
consuming.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1016&gt;
GRASS GIS <http://grass.osgeo.org>

#1016: v.info support for level 1 (patch)
--------------------------+-------------------------------------------------
  Reporter: neteler | Owner: grass-dev@lists.osgeo.org
      Type: enhancement | Status: new
  Priority: normal | Milestone: 6.4.1
Component: Vector | Version: svn-develbranch6
Resolution: | Keywords: v.info
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Changes (by hamish):

  * keywords: => v.info

Comment:

is it not possible to have the level automatically detected instead of
requiring to use an extra flag?

If you had level 2 topo, I don't see why you'd want to force opening at
level 1 for v.info.

thanks,
Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1016#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

#1016: v.info support for level 1 (patch)
--------------------------+-------------------------------------------------
  Reporter: neteler | Owner: grass-dev@lists.osgeo.org
      Type: enhancement | Status: new
  Priority: normal | Milestone: 6.4.1
Component: Vector | Version: svn-develbranch6
Resolution: | Keywords: v.info
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Comment (by mmetz):

Replying to [comment:1 hamish]:
> is it not possible to have the level automatically detected instead of
requiring to use an extra flag?
>
> If you had level 2 topo, I don't see why you'd want to force opening at
level 1 for v.info.

Right. v.info no longer needs level 2, it tries level 2 first, if this is
not successful, it tries level 1.

There is also no need to force level 1 just to speed up things, because
v.info opens vectors with head only when trying level 2. At least this is
the original behaviour in 6.4 and earlier, it was changed recently for 6.5
and 7, I changed it back to head only. There is no need to fully open a
vector if level 2 is available.

In short, whatever happens, v.info is lightening fast and uses nearly no
memory. The level 1 option can go.

Um, v.info is fast on cli but slow when used within wxGUI with a large
vector, but that's a wxGUI thing, wxGUI collects some vector information
first before it allows you to run a module.

Markus M

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1016#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

#1016: v.info support for level 1 (patch)
-------------------------+--------------------------------------------------
Reporter: neteler | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.1
Component: Vector | Version: svn-develbranch6
Keywords: v.info | Platform: All
      Cpu: All |
-------------------------+--------------------------------------------------

Comment(by hamish):

now fixed in all branches?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1016#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>

#1016: v.info support for level 1 (patch)
--------------------------+-------------------------------------------------
  Reporter: neteler | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 6.4.1
Component: Vector | Version: svn-develbranch6
Resolution: fixed | Keywords: v.info
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Changes (by mmetz):

  * status: new => closed
  * resolution: => fixed

Comment:

Replying to [comment:3 hamish]:
> now fixed in all branches?

Yes, closing ticket on behalf of Markus N.

Markus M

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1016#comment:4&gt;
GRASS GIS <http://grass.osgeo.org>