[GRASS-dev] Re: [GRASS GIS] #498: r.sun2 commissioning trials

[GRASS GIS]

#498: r.sun2 commissioning trials
---------------------+------------------------------------------------------
  Reporter:  hamish  |       Owner:  hamish          
      Type:  defect  |      Status:  assigned        
  Priority:  major   |   Milestone:  6.4.0           
 Component:  Raster  |     Version:  svn-develbranch6
Resolution:          |    Keywords:  r.sun           
  Platform:  All     |         Cpu:  All             
---------------------+------------------------------------------------------
Changes (by hamish):

  * platform:  Linux => All
  * cpu:  x86-32 => All

Comment:

 Hi,

 lat=, latin=, and longin= now play nice with pj_proj() in devbr6 and
 trunk.

  - lat= can no longer work due to rsunlib's needs, so I ripped it out.
 It's kind of lame anyway.
  - latin= and longin= need to be given together (except with civiltime=)
  - even if you give those seed raster maps it only gives you about a 1.3%
 speed increase (4 sec over 4.5minutes for my single test). The calc in
 rsunlib needs Cartesian coords so it only really saves the pj_proj() calc
 in one place, and that really isn't so expensive anyway. The man page says
 those lat= opts are there in case you want to run r.sun from a simple XY
 location. With the new pj_proj() stuff in rsunlib I don't think that's
 viable anymore, and even if it did work it's not worth the confusion it
 introduces. Given all that I'm inclined to rip out those options and have
 civiltime= simply use pj_proj() if it wants to know the local longitude.

 see r38770,2.


 Hamish

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/498#comment:14>
GRASS GIS <http://grass.osgeo.org>