Opened 9 years ago

Closed 9 years ago

#1016 closed enhancement (fixed)

v.info support for level 1 (patch)

Reported by: neteler Owned by: grass-dev@…
Priority: normal Milestone: 6.4.1
Component: Vector Version: svn-develbranch6
Keywords: v.info Cc:
CPU: All Platform: All

Description

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.

Attachments (1)

v.info.diff (5.9 KB) - added by neteler 9 years ago.
backport patch

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by neteler

Attachment: v.info.diff added

backport patch

comment:1 Changed 9 years ago by hamish

Keywords: v.info added

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

comment:2 in reply to:  1 Changed 9 years ago by mmetz

Replying to 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

comment:3 Changed 9 years ago by hamish

now fixed in all branches?

comment:4 in reply to:  3 Changed 9 years ago by mmetz

Resolution: fixed
Status: newclosed

Replying to hamish:

now fixed in all branches?

Yes, closing ticket on behalf of Markus N.

Markus M

Note: See TracTickets for help on using tickets.