Ticket #1584 (new defect)

Opened 15 months ago

Last modified 9 months ago

error in r.walk help page

Reported by: hamish Owned by: grass-dev@…
Priority: major Milestone: 6.4.3
Component: Docs Version: svn-releasebranch64
Keywords: r.walk Cc: Agustin.Lobo@…
Platform: All CPU: 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

in reply to: ↑ description ; follow-up: ↓ 2   Changed 12 months ago by 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: ↑ 1   Changed 12 months ago by dkavanagh

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   Changed 12 months ago by dkavanagh

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

  Changed 9 months ago by neteler

  • version changed from unspecified to svn-releasebranch64
Note: See TracTickets for help on using tickets.