[GRASS-dev] r.le bugs--list needed

[Brad Douglas]

I completely read the original message wrong.  I thought you were making
a bug report for some reason.  Doh! :-)  I hadn't realized you were the
author.

I believe the usages of stderr, stdout,etc. are proper, but there's a
push to get the modules to use G_message(), G_warning(),
G_fatal_error(), etc.  It'll be a large task that will eventually need
to be done.

If you'd like some help, I can assist.  I'll go over the source again
and send you an email off list.  If you'll be actively working on it,
there's no need to remove it from CVS.

On Mon, 2006-09-25 at 15:43 -0600, William L. Baker wrote:
> Hello Brad,
> 
> Thank you for your thoughts. I wrote the code with assistance, so have
> some advantages there. But, disadvantage is that I have not followed
> GRASS updates since 5.2 or so. For example, I did not know there was a
> problem with using stderr, stdout, or files. Can someone explain what
> should be done instead? 
> 
> Yes, it could be removed from the production branches if you like.
> Until I work on it, there probably won't be fixes. But, if someone who
> knows GRASS revisions could help, then maybe this could be done more
> quickly? But, eventually I would like it to work, and I will slowly
> get up to speed and fix it even if no one has time (with some
> occasional help).
> 
> One thing that would help me right away would be if someone would
> quickly scan the source code to see any obvious inconsistencies with
> 6.xxx.
> 
> r.le.dist was never ported to the 5.XXX revision, as I ran out of time
> to do that. So, it would require a much larger revision. 
> 
> r.le.patch, r.le.setup, and r.le.pixel I think should not be so hard.
> 
> Bill Baker, Univ. of Wyoming
> 
> 
> 
> Brad Douglas wrote: 
> > On Mon, 2006-09-25 at 14:51 -0600, William L. Baker wrote:
> >   
> > > Hello,
> > > 
> > > I may (still uncertain!) slowly try to get r.le programs to work under 
> > > 6.X. Here is what I see as problems to be fixed. If anyone knows of 
> > > other problems, let me know:
> > > 
> > > 1. r.le.setup fails when a sitefile is requested--update needed
> > > 2. r.le.setup does not display properly in the display window
> > > 3. with gis.m, r.le.setup does not connect to display window
> > > 4. r.le.pixel crashes with a memory problem
> > > 5. no r.le.dist program
> > >     
> > 
> > William,
> > 
> > r.le needs major work.  It has not been touched in several years and I'm
> > a bit surprised that it still works at all.  I got a start on it last
> > week by at least resolving compiler errors/warnings for r.le.patch, but
> > I didn't really attempt to fix anything.
> > 
> > Is there anyone who has the time to deal with r.le in the near term?  It
> > will be a fairly time consuming project if you plan on updating text I/O
> > (it references stderr, stdout, and files extensively).
> > 
> > What happened to r.le.dist, anyway?  Would it be best to remove r.le
> > from production branches (6.2) until it works reliably again?


-- 
Brad Douglas <rez touchofmadness com>                      KB8UYR
Address: 37.493,-121.924 / WGS84    National Map Corps #TNMC-3785