Opened 7 months ago

Closed 7 months ago

Last modified 7 months ago

#3846 closed defect (fixed)

r.contour started producing broken/segmented contour lines since 7.4

Reported by: metello Owned by: grass-dev@…
Priority: normal Milestone: 7.6.2
Component: Raster Version: unspecified
Keywords: r.contour Cc:
CPU: x86-64 Platform: MSWindows

Description

Since GRASS 7.4, some contour lines generated by r.contour are broken into several segments (until 7.2 was ok). I have a system that uses r.contour to trace reservoirs for hydroelectric plants and that problem is breaking my algorithm. In the attached GRASS project, you can see the result generated by different GRASS versions (contour level = 229) - look at the largest contour line. I tested the following versions: 7.2.3, 7.4.4, 7.6 and 7.7svn. Only 7.2.3 produced a continuous/unbroken contour.

Change History (6)

comment:1 Changed 7 months ago by metello

The file was too large to attach. I uploaded here: http://psr-shared.s3.amazonaws.com/public/admin/2C66EC59/R_CONTOUR_BUG.zip

Coordinates of some points of discontinuity: (384073.870002,8827636.45371), (402242.43372,8799021.33681), (409632.732253,8796197.58953)

comment:2 Changed 7 months ago by metello

Keywords: ​helpwanted added

comment:3 in reply to:  1 ; Changed 7 months ago by mmetz

Replying to metello:

The file was too large to attach. I uploaded here: http://psr-shared.s3.amazonaws.com/public/admin/2C66EC59/R_CONTOUR_BUG.zip

Coordinates of some points of discontinuity: (384073.870002,8827636.45371), (402242.43372,8799021.33681), (409632.732253,8796197.58953)

I can confirm the missing segments, the previous fix to prevent contours from tracing back on themselves was not working. Working on it...

comment:4 in reply to:  3 Changed 7 months ago by mmetz

Replying to mmetz:

Replying to metello:

The file was too large to attach. I uploaded here: http://psr-shared.s3.amazonaws.com/public/admin/2C66EC59/R_CONTOUR_BUG.zip

Coordinates of some points of discontinuity: (384073.870002,8827636.45371), (402242.43372,8799021.33681), (409632.732253,8796197.58953)

I can confirm the missing segments, the previous fix to prevent contours from tracing back on themselves was not working. Working on it...

r.contour has been fixed in master, relbr76, and relbr74.

comment:5 Changed 7 months ago by neteler

Keywords: ​helpwanted removed
Milestone: 7.6.2
Resolution: fixed
Status: newclosed

Closing as fixed.

comment:6 Changed 7 months ago by martinl

Note: See TracTickets for help on using tickets.