Opened 12 years ago

Closed 9 years ago

Last modified 8 years ago

#1584 closed defect (fixed)

error in r.walk help page

Reported by: hamish Owned by: grass-dev@…
Priority: major Milestone: 7.2.0
Component: Docs Version: svn-releasebranch64
Keywords: r.walk Cc: Agustin.Lobo@…
CPU: All Platform: All

Description

one from Agustin:

the documentation in r.walk is wrong. It says:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, 1.9998, -1.9998)"
and it should say:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, -1.9998, 1.9998)"

because the point is that you go faster on gentle slopes and more
slowly on steeper slopes and those values are in m/s
The worrying question is whether these values are also wrong in the
default values of r.walk.

Change History (7)

in reply to:  description ; comment:1 by dkavanagh, 12 years ago

There is also a mistake in the Standard Directions figure in the Movement Direction section. 90 should be immediately below x and the zero should be removed.

Replying to hamish:

one from Agustin:

the documentation in r.walk is wrong. It says:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, 1.9998, -1.9998)"
and it should say:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, -1.9998, 1.9998)"

because the point is that you go faster on gentle slopes and more
slowly on steeper slopes and those values are in m/s
The worrying question is whether these values are also wrong in the
default values of r.walk.

in reply to:  1 comment:2 by dkavanagh, 12 years ago

My note applies to 6.5 and 7.0. The Langmuir correction applies to all versions.

Replying to dkavanagh:

There is also a mistake in the Standard Directions figure in the Movement Direction section. 90 should be immediately below x and the zero should be removed.

Replying to hamish:

one from Agustin:

the documentation in r.walk is wrong. It says:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, 1.9998, -1.9998)"
and it should say:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, -1.9998, 1.9998)"

because the point is that you go faster on gentle slopes and more
slowly on steeper slopes and those values are in m/s
The worrying question is whether these values are also wrong in the
default values of r.walk.

in reply to:  description comment:3 by dkavanagh, 12 years ago

Hamish - I believe the help is correct. c and d are multiplied by the delta, which in a downward slope will be negative. Here is an example from the code - fcost_dtm = (double)((double)(W_dtm - my_dtm) * c);

Replying to hamish:

one from Agustin:

the documentation in r.walk is wrong. It says:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, 1.9998, -1.9998)"
and it should say:
"The default values for a, b, c, d are those proposed by Langmuir
(0.72, 6.0, -1.9998, 1.9998)"

because the point is that you go faster on gentle slopes and more
slowly on steeper slopes and those values are in m/s
The worrying question is whether these values are also wrong in the
default values of r.walk.

comment:4 by neteler, 12 years ago

Version: unspecifiedsvn-releasebranch64

comment:5 by neteler, 9 years ago

Milestone: 6.4.37.1.0

Bumped for check to 7.1. In case backports down to G6.4 are needed.

in reply to:  5 comment:6 by annakrat, 9 years ago

Resolution: fixed
Status: newclosed

Replying to neteler:

Bumped for check to 7.1. In case backports down to G6.4 are needed.

I was looking into that recently in 7 and I think it's fine. I backported some of the documentation in r65411.

comment:7 by neteler, 8 years ago

Milestone: 7.1.07.2.0

Milestone renamed

Note: See TracTickets for help on using tickets.