Opened 14 years ago

Closed 11 years ago

Last modified 11 years ago

#1241 closed enhancement (fixed)

Change default calculation in r.sun to include topographic shadowing

Reported by: newcombttg Owned by: hamish
Priority: normal Milestone: 7.0.0
Component: Raster Version: svn-trunk
Keywords: r.sun Cc: grass-dev@…
CPU: All Platform: All


The r.sun command has a -s option to incorporate the shadowing effect of terrain on the calculations. Request that this be made the default. I understand that there are circumstances where one would not want the terrain shadowing to be used, but it seems that incorporating terrain shadowing as the default would give the more commonly desired outcome.

Change History (5)

comment:1 by hamish, 14 years ago

Cc: grass-dev@… added
Keywords: r.sun added
Owner: changed from grass-dev@… to hamish
Status: newassigned
Version: unspecifiedsvn-trunk

comment:2 by hamish, 11 years ago

any votes on the matter from the user community?

comment:3 by hamish, 11 years ago

Resolution: fixed
Status: assignedclosed

Yann and MarkusN like it too; done in r55187.

There is now a '-p' planar flag. (if you like a better non-"s" letter, by all means change it..)

Also I expanded some option names to take advantage of the new grass7 underscore option name parsing.


comment:5 by hamish, 11 years ago

see also the "Renamed options" section further down the page, already done. :)


Note: See TracTickets for help on using tickets.