Opened 3 years ago

Closed 3 years ago

Last modified 3 years 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 by metello, 3 years ago

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 by metello, 3 years ago

Keywords: ​helpwanted added

in reply to:  1 ; comment:3 by mmetz, 3 years ago

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...

in reply to:  3 comment:4 by mmetz, 3 years ago

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 by neteler, 3 years ago

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

Closing as fixed.

Note: See TracTickets for help on using tickets.