Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#3797 closed defect (fixed)

r.horizon suffix gets messed up in multi-direction mode

Reported by: sbl Owned by: grass-dev@…
Priority: normal Milestone: 7.8.0
Component: Raster Version: svn-trunk
Keywords: r.horizon Cc:
CPU: Unspecified Platform: Linux


Change History (5)

comment:1 by mmetz, 5 years ago

Fixed in trunk r74298.

in reply to:  2 comment:3 by mmetz, 5 years ago

comment:4 by sbl, 5 years ago

Resolution: fixed
Status: newclosed

Very good. Then I take the liberty to close this ticket...

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

Replying to sbl:

Very good. Then I take the liberty to close this ticket...

Hi Stefan, you are the original reporter of this ticket, therefore you have IMHO the right to close the ticket. Someone else submitting a fix can also close the ticket if this person is really confident that the issue has been fixed.

In this case, my system behaved differently: the wrong output on was "test_horizon_output_from_elevation_%03", but on my system it was "test_horizon_output_from_elevation_", i.e. my system behaved differently. With my fix, my system produced correct output, and I was quite but not really sure that my fix works on other systems as well. Thus waiting for confirmation by the original reporter that the fix is working on other systems as well.

This particular bug and the related fix passed CI, thus CI is in this case not good enough, only the test for r.horizon. Another point to

  1. get the testsuite all green
  2. make CI run the testsuite and let CI fail if the testsuite is not all green
Note: See TracTickets for help on using tickets.