Opened 9 years ago

Closed 6 years ago

Last modified 6 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

Description

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 Changed 9 years ago by hamish

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

comment:2 Changed 6 years ago by hamish

any votes on the matter from the user community?

comment:3 Changed 6 years ago by hamish

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.

Hamish

comment:5 Changed 6 years ago by hamish

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

https://trac.osgeo.org/grass/wiki/Grass7/NewFeatures?action=diff&version=143

H

Note: See TracTickets for help on using tickets.