Opened 8 years ago

Last modified 8 years ago

#2941 closed defect

r.sun: wrong sign in equation of time — at Version 1

Reported by: tomhuld Owned by: grass-dev@…
Priority: normal Milestone: 7.0.4
Component: Raster Version: svn-releasebranch70
Keywords: r.sun Cc:
CPU: Unspecified Platform: Unspecified

Description (last modified by neteler)

Bug report ticket #2876 by fabriziosossan reported a problem with the equation of time in r.sun, whereby the calculation of the solar time from the clock (civil) time is wrong. The problem was fixed with a patch for GRASS 6, but as far as I know there the fix was not applied to GRASS 7.

I would like to suggest the following patch for r.sun/main.c. Since this is my first patch for GRASS, apologies if I'm doing it worng.

--- main.c      (revision 67790)
+++ main.c      (working copy)
@@ -1766,7 +1766,7 @@
         * "local clock time". */
        dayRad = 2. * M_PI * day / 365.25;
        locTimeOffset =
-           -0.128 * sin(dayRad - 0.04887) - 0.165 * sin(2 * dayRad +
+           +0.128 * sin(dayRad - 0.04887) + 0.165 * sin(2 * dayRad +
                                                         0.34383);
 
        /* Time offset due to timezone as input by user */

Change History (1)

comment:1 by neteler, 8 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.