<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Hamish,<br>
<br>
Great. I will get the CVS HEAD version next and have a look. Will try
r.le.pixel with sample dataset. I will see if there are other major
problems that prevent some use of r.le for now, until r.li is fully
developed. <br>
<br>
r.le.para, r.le.null, and r.le.rename are not very important programs
and for now they can be dropped. <br>
r.le.dist is something I should update to 6.XXX, but that will take
quite some time (months). <br>
<br>
I will leave it to GRASS developers to decide whether to keep r.le for
now or remove it until it is fixed or r.li is ready. If it is removed,
I can work on it with some help from you all and then put it back in
once it is ready. If it is kept in, maybe Hamish and I and some others
can make sure it generally works. <br>
<br>
Thank you!<br>
<br>
Bill Baker<br>
<br>
Hamish wrote:<br>
<blockquote cite="mid20060927184857.1628e245.hamish_nospam@yahoo.com"
type="cite">
<pre wrap="">William L. Baker wrote:
</pre>
<blockquote type="cite">
<pre wrap="">I may (still uncertain!) slowly try to get r.le programs to work under
6.X.
</pre>
</blockquote>
<pre wrap=""><!---->
The raster format & GIS API is unchanged, only vector & sites have
changed. So required fixes for porting a raster module are in fact
small.
</pre>
<blockquote type="cite">
<pre wrap="">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
</pre>
</blockquote>
<pre wrap=""><!---->
fixed in CVS HEAD + 6.2 branch
- just removed the prompt
In CVS HEAD I've changed the module to use G_parser() instead of custom
prompts.
</pre>
<blockquote type="cite">
<pre wrap="">2. r.le.setup does not display properly in the display window
</pre>
</blockquote>
<pre wrap=""><!---->
fixed in CVS HEAD + 6.2 branch
- it was hanging on G_system(" d.frame -e");
changed to use D_setup(1) to clear the screen
</pre>
<blockquote type="cite">
<pre wrap="">3. with gis.m, r.le.setup does not connect to display window
</pre>
</blockquote>
<pre wrap=""><!---->
fixed in CVS HEAD + 6.2 branch
- menu.tcl needed "guarantee_xmon;"
</pre>
<blockquote type="cite">
<pre wrap="">4. r.le.pixel crashes with a memory problem
</pre>
</blockquote>
<pre wrap=""><!---->
Brad?
William: Can you prepare an example using the spearfish dataset? These
modules are too complex for me to quickly understand.
</pre>
<blockquote type="cite">
<pre wrap="">5. no r.le.dist program
</pre>
</blockquote>
<pre wrap=""><!---->
Also, no r.le.para as exists in GRASS 5, nor r.le.null & r.le.rename as
mentioned in the help page. If these should be copied from GRASS 5, let
us know, it's not hard to do. I don't see r.le.dist built in GRASS 5.4
either??
I notice that the GRASS 5 r.le man pages were not ported into GRASS 6
description.html files. That's easy to do and should be done.
Brad:
</pre>
<blockquote type="cite">
<pre wrap="">Would it be best to remove r.le from production branches (6.2) until
it works reliably again?
</pre>
</blockquote>
<pre wrap=""><!---->
If r.le is deemed not ready for release, removing it is as simple as
editing raster/Makefile and removing the entries for it in menu.tcl.
Markus:
</pre>
<blockquote type="cite">
<pre wrap="">please note that there are the r.li efforts:
<a class="moz-txt-link-freetext" href="http://www.faunalia.it/mediawiki/index.php/R_li">http://www.faunalia.it/mediawiki/index.php/R_li</a>
Maybe efforts could be joined?
</pre>
</blockquote>
<pre wrap=""><!---->
Yes please.. but it is simple to copy over missing r.le modules and man
pages, it can be done in a few minutes. So we should do that while we wait
for r.li to be ready.
Hamish
</pre>
</blockquote>
</body>
</html>