[GRASS-dev] [GRASS GIS] #1686: v.mkgrid should offer breaks option for vertical lines too

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: unspecified
Keywords: mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
Issue mentioned in user ML by Hamish.

Design decision needed - should breaks be used for both horizontal and
vertical breaks or it should be renamed to h_breaks and v_breaks?

Original message:
20deg reprojected grid lines are a bit ugly, apparently we
need to extend the v.mkgrid breaks= option to work on vertical
lines as well as the horizontal ones.
http://lists.osgeo.org/pipermail/grass-user/2012-July/065214.html

Image displaying the issue: http://lists.osgeo.org/pipermail/grass-
user/attachments/20120702/066007f4/attachment.png

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

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: unspecified
Keywords: v.mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by neteler):

  * keywords: mkgrid => v.mkgrid

Comment:

Replying to [ticket:1686 marisn]:
...
> 20deg reprojected grid lines are a bit ugly, apparently we
> need to extend the v.mkgrid breaks= option to work on vertical
> lines as well as the horizontal ones.
> http://lists.osgeo.org/pipermail/grass-user/2012-July/065214.html
>
> Image displaying the issue: http://lists.osgeo.org/pipermail/grass-
user/attachments/20120702/066007f4/attachment.png

Anyone able to extend the breaks to vertical lines in v.mkgrid? Thanks...

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

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: unspecified
Keywords: v.mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by lucadelu):

Replying to [comment:1 neteler]:

>
> Anyone able to extend the breaks to vertical lines in v.mkgrid?
Thanks...

I added breaks to vertical lines to v.mkgrid, please check the attached
patch.

In the screenshots you can see:
  * red lines: old behavior of v.mkgrid
  * blue lines: new behavior of v.mkgrid with default breaks value (3)
  * green lines: new behavior of v.mkgrid with breaks value 30

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

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: unspecified
Keywords: v.mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by lucadelu):

Replying to [comment:2 lucadelu]:

>
> I added breaks to vertical lines to v.mkgrid, please check the attached
patch.
>
> In the screenshots you can see:
> * red lines: old behavior of v.mkgrid
> * blue lines: new behavior of v.mkgrid with default breaks value (3)
> * green lines: new behavior of v.mkgrid with breaks value 30

I committed the changes in r61552. Please try it

It should be backported to release branch

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

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch70
Keywords: v.mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by neteler):

  * version: unspecified => svn-releasebranch70

Comment:

Tested -l ok, thanks for the fixes and improvements. See attached
screenshot of a
v.mkgrid result generated in EPSG 3035, then reprojected to Latlong EPSG
4326.

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

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch70
Keywords: v.mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by neteler):

Note: this vertex densification should also be added to v.proj

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

#1686: v.mkgrid should offer breaks option for vertical lines too
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch70
Keywords: v.mkgrid | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by neteler):

Replying to [comment:5 neteler]:
> Note: this vertex densification should also be added to v.proj

Thanks to Markus Metz that got implemented in v.proj in r63307 (trunk).

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

#1686: v.mkgrid should offer breaks option for vertical lines too
--------------------------+-------------------------------------------------
  Reporter: marisn | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch70
Resolution: fixed | Keywords: v.mkgrid
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by neteler):

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

Comment:

All implemented, working fine. Closing.

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