[GRASS-dev] Re: [GRASS GIS] #1006: r.terraflow fails to stat() stream file on Windows

[GRASS GIS]

#1006: r.terraflow fails to stat() stream file on Windows
-----------------------------------+----------------------------------------
 Reporter:  marisn                 |       Owner:  grass-dev@…              
     Type:  defect                 |      Status:  new                      
 Priority:  critical               |   Milestone:  6.4.0                    
Component:  Raster                 |     Version:  svn-releasebranch64      
 Keywords:  wingrass, r.terraflow  |    Platform:  MSWindows Vista          
      Cpu:  x86-32                 |  
-----------------------------------+----------------------------------------

Comment(by hamish):

 Replying to [comment:35 mmetz]:
 > the ability to process large datasets is also covered by other modules.

 I know that, but the (perhaps recently historic) way the modules are
 presented in the menus does not make that clear to users.

 > Tested 6.5svn r42716 on XP, same error as in comment:29 in filldepr.cc.

 thanks. so the error is now:
 {{{
 Reading data from <elevation> to stream
 <C:/Users/syringia/AppData/Local/Temp/STREAM_a06204>
 [...]
 Assertion failed: ae == AMI_ERROR_NO_ERROR, file
 filldepr.cc, line 136
 This application has requested the Runtime to terminate it
 in an unusual way.
 }}}

 from raster/r.terraflow/filldepr.cc
 {{{
     /*read next edge*/
     ae = boundaryStr->read_item(&nextedge);
     assert(ae == AMI_ERROR_NO_ERROR);
 }}}

 which calls include/iostream/mem_stream.h
 {{{
 template<class T>
 AMI_err MEM_STREAM<T>::read_item(T **elt)  {

   assert(data);

   if(curr == dataend) {
     return AMI_ERROR_END_OF_STREAM;
   }
   *elt = curr;
   curr++;
   return AMI_ERROR_NO_ERROR;
 };
 }}}

 so it seems that it is prematurely reaching the end of the stream?

 > Once that is fixed, we might have a look at why r.terraflow refuses to
 run with output from
 > r.volcano...

 does it not?? how about a sum=r.volcano+r.surf.fractal surface? perhaps
 because there are no basins except the small one in the crater in that
 fake landscape? is there a ticket for this?


 > No, seriously, r.terraflow and iostream is written for Linux, as also
 previous tickets have
 > shown, no reason to expect that this is the last problem with Windows.
 > Please leave it for 6.5 or 7.0.

 we could easily say the same for all of GRASS. some parts will be easier
 to port than others, but if we didn't care about MS-Windows at all we
 could have released 6.4 mid-last year. in this case it sucks, is not known
 to be fundamentally deep, and I'd like to try and fix it. even if it is
 crippled due to LFS (you might say 32bit XP is too with its 3.2gb RAM
 limit), at least people can still work on tutorials/examples and learn how
 to use it for when they graduate to a gruntier system.

 in pursuit of a technical fix I will add debug messages to mem_stream.h.
 (r42731)


 regards,
 Hamish

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/1006#comment:36>
GRASS GIS <http://grass.osgeo.org>