[GRASS-dev] GRASS 6.2.2 release candidate preparation

Markus_Neteler1 · May 8, 2007, 8:57am

Hi,

time has passed and we have fixed more problems in GRASS 6.2-CVS:
http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#Fixed

I feel that we should get out RC1 this week to avoid further
delay. The TODO list shows:

* gis.m: any needed backports??
  -> no idea about this
* NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)
    [remove G_warning() from nviz/src/nviz_init.c if tests ok]
    http://freegis.org/cgi-bin/viewcvs.cgi/grass6/visualization/nviz/src/map_obj.c
    still not fully fixed?? nothing backported yet.
  -> I have backported this fix, it seems to cure the problem

So, only gis.m remains *in case there is anything to do*.

Markus

H_B · May 8, 2007, 9:54am

Markus Neteler wrote:

* NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)

..

-> I have backported this fix, it seems to cure the problem

not 100% cured, see end of the bug report:
https://intevation.de/rt/webrt?display=History&serial_num=4725

G622> nviz vol=vox50
#loads ok (see bug report for generating vox50)
# view height -> 75.0
# Visualize -> Volumes
# Isosurface [Add]
Segmentation fault

Try again, then it works ok.

Same thing if started with "nviz -q" then load [New] Raster Volume.
First time you add a new isosurface it segfaults, if you restart nviz
then it works. Smells like a little uninit'd memory error remains.

Previously, AFAIK, it didn't segfault if you started with "nviz -q".
Apparently the fix makes the command line way better, but the manual
add way worse. ??

Hamish

Markus_Neteler1 · May 8, 2007, 10:08am

On Tue, May 08, 2007 at 09:54:31PM +1200, Hamish wrote:

Markus Neteler wrote:
> * NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)
..
> -> I have backported this fix, it seems to cure the problem

not 100% cured, see end of the bug report:
https://intevation.de/rt/webrt?display=History&serial_num=4725

G622> nviz vol=vox50
#loads ok (see bug report for generating vox50)
# view height -> 75.0
# Visualize -> Volumes
# Isosurface [Add]
Segmentation fault

Try again, then it works ok.

Confirmed.

Same thing if started with "nviz -q" then load [New] Raster Volume.
First time you add a new isosurface it segfaults, if you restart nviz
then it works. Smells like a little uninit'd memory error remains.

Previously, AFAIK, it didn't segfault if you started with "nviz -q".
Apparently the fix makes the command line way better, but the manual
add way worse. ??

Well, I dunno...
I just feel that we should not delay RC1 forever.

Markus

cmbarton · May 8, 2007, 3:08pm

The only thing I can think of is if you want to backport the font selection
utility I've added. If so, it should probably be backported as is right now.
Currently, it uses a TclTk file browser to pick a font to use with
GRASS_FONT. I am about to change this to a scrolling list control that will
pick a font from the list generated by d.font -l.

BUT, I don't think this new method will work with the rest of 6.2.2 because
it needs recent updates to the font drivers and Glynn's new mkftcap script.
So 6.2.2 should stick with the file selection approach.

I also went through recently (several weeks back) and added catch clauses to
all open and close statements to make the GUI more robust against crashing.
This affected a large portion of the TclTk modules. I don't know if this was
backported or not.

Michael

On 5/8/07 1:57 AM, "Markus Neteler" <neteler@itc.it> wrote:

Hi,

time has passed and we have fixed more problems in GRASS 6.2-CVS:
http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#Fixed

I feel that we should get out RC1 this week to avoid further
delay. The TODO list shows:

* gis.m: any needed backports??
  -> no idea about this
* NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)
    [remove G_warning() from nviz/src/nviz_init.c if tests ok]

FreeGIS.org
    still not fully fixed?? nothing backported yet.
  -> I have backported this fix, it seems to cure the problem

So, only gis.m remains *in case there is anything to do*.

Markus

__________________________________________
Michael Barton, Professor of Anthropology
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

phone: 480-965-6213
fax: 480-965-7671
www: http://www.public.asu.edu/~cmbarton

Markus_Neteler1 · May 8, 2007, 4:01pm

Michael Barton wrote on 05/08/2007 05:08 PM:

The only thing I can think of is if you want to backport the font selection
utility I've added. If so, it should probably be backported as is right now.
Currently, it uses a TclTk file browser to pick a font to use with
GRASS_FONT. I am about to change this to a scrolling list control that will
pick a font from the list generated by d.font -l.

BUT, I don't think this new method will work with the rest of 6.2.2 because
it needs recent updates to the font drivers and Glynn's new mkftcap script.
So 6.2.2 should stick with the file selection approach.

I would prefer to not do that. At least not for 6.2.2.

I also went through recently (several weeks back) and added catch clauses to
all open and close statements to make the GUI more robust against crashing.
This affected a large portion of the TclTk modules. I don't know if this was
backported or not.

I don't know - I guess not. Looks like this list:
2007-04-30 02:57 michael
* chart.tcl, gmtree.tcl, thematic.tcl: Improved error trapping by
adding catch clauses to open and close statements

2007-04-30 02:48 michael
* maplabels.tcl: Added catch clause to open and close statments for
better error trapping. Update some other features.

2007-04-30 02:47 michael
    * mapprint.tcl, raster.tcl, tksys.tcl, vector.tcl: Improved error
      trapping by adding or enhancing catch clauses on open and close
      statements

2007-04-30 01:52 michael
    * georect.tcl: Added error traps (catch clauses) to open and close
      statements. catch on close will print error to terminal
      (hopefully).

Also fixed serious bug caused by new map@mapset syntax for maps
in current mapset. This had broken georect completely.

2007-04-30 01:51 michael
    * gm.tcl, histogram.tcl, rastnums.tcl, runandoutput.tcl: Added
      error traps (catch clauses) to open and close statements. catch
      on close will print error to terminal (hopefully).

2007-04-29 22:48 michael
* rastnums.tcl: Added catch to close statement for better error
trapping

2007-04-29 22:48 michael
* georect.tcl: Added catch to close statement (following open
g.region) for better error trapping

Markus

------------------
ITC -> dall'1 marzo 2007 Fondazione Bruno Kessler
ITC -> since 1 March 2007 Fondazione Bruno Kessler
------------------

cmbarton · May 8, 2007, 5:37pm

On 5/8/07 9:01 AM, "Markus Neteler" <neteler@itc.it> wrote:

Michael Barton wrote on 05/08/2007 05:08 PM:

The only thing I can think of is if you want to backport the font selection
utility I've added. If so, it should probably be backported as is right now.
Currently, it uses a TclTk file browser to pick a font to use with
GRASS_FONT. I am about to change this to a scrolling list control that will
pick a font from the list generated by d.font -l.

BUT, I don't think this new method will work with the rest of 6.2.2 because
it needs recent updates to the font drivers and Glynn's new mkftcap script.
So 6.2.2 should stick with the file selection approach.

I would prefer to not do that. At least not for 6.2.2.

I also went through recently (several weeks back) and added catch clauses to
all open and close statements to make the GUI more robust against crashing.
This affected a large portion of the TclTk modules. I don't know if this was
backported or not.

Yes, this is the list. It involves a lot of modules.

Michael

I don't know - I guess not. Looks like this list:
2007-04-30 02:57 michael
    * chart.tcl, gmtree.tcl, thematic.tcl: Improved error trapping by
      adding catch clauses to open and close statements

2007-04-30 02:48 michael
    * maplabels.tcl: Added catch clause to open and close statments for
      better error trapping. Update some other features.

2007-04-30 02:47 michael
    * mapprint.tcl, raster.tcl, tksys.tcl, vector.tcl: Improved error
      trapping by adding or enhancing catch clauses on open and close
      statements

2007-04-30 01:52 michael
    * georect.tcl: Added error traps (catch clauses) to open and close
      statements. catch on close will print error to terminal
      (hopefully).

      Also fixed serious bug caused by new map@mapset syntax for maps
      in current mapset. This had broken georect completely.

2007-04-30 01:51 michael
    * gm.tcl, histogram.tcl, rastnums.tcl, runandoutput.tcl: Added
      error traps (catch clauses) to open and close statements. catch
      on close will print error to terminal (hopefully).

2007-04-29 22:48 michael
    * rastnums.tcl: Added catch to close statement for better error
      trapping

2007-04-29 22:48 michael
    * georect.tcl: Added catch to close statement (following open
      g.region) for better error trapping

Markus

------------------
ITC -> dall'1 marzo 2007 Fondazione Bruno Kessler
ITC -> since 1 March 2007 Fondazione Bruno Kessler
------------------

__________________________________________
Michael Barton, Professor of Anthropology
School of Human Evolution & Social Change
Center for Social Dynamics and Complexity
Arizona State University

phone: 480-965-6213
fax: 480-965-7671
www: http://www.public.asu.edu/~cmbarton

Maciej_Sieczka2 · May 9, 2007, 3:58pm

Markus Neteler wrote:

Hi,

time has passed and we have fixed more problems in GRASS 6.2-CVS:
http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#Fixed

I feel that we should get out RC1 this week to avoid further
delay. The TODO list shows:

* gis.m: any needed backports??
  -> no idea about this
* NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)
    [remove G_warning() from nviz/src/nviz_init.c if tests ok]
    http://freegis.org/cgi-bin/viewcvs.cgi/grass6/visualization/nviz/src/map_obj.c
    still not fully fixed?? nothing backported yet.
  -> I have backported this fix, it seems to cure the problem

Sorry, I don't have time to check thoroughly, but among serious bugs
remaining in 6.2.2 CVS, fixed in 6.3 CVS, these come to my mind at a
glance:

GForge #343 v.clean tool=snap
RT #4247 v.build.polylines
RT #4249 v.build.polylines

Maciek

Markus_Neteler1 · May 9, 2007, 4:03pm

On Wed, May 09, 2007 at 05:58:18PM +0200, Maciej Sieczka wrote:

Markus Neteler wrote:
> Hi,
>
> time has passed and we have fixed more problems in GRASS 6.2-CVS:
> http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#Fixed
>
> I feel that we should get out RC1 this week to avoid further
> delay. The TODO list shows:
>
> * gis.m: any needed backports??
> -> no idea about this
> * NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)
> [remove G_warning() from nviz/src/nviz_init.c if tests ok]
> http://freegis.org/cgi-bin/viewcvs.cgi/grass6/visualization/nviz/src/map_obj.c
> still not fully fixed?? nothing backported yet.
> -> I have backported this fix, it seems to cure the problem

Sorry, I don't have time to check thoroughly, but among serious bugs
remaining in 6.2.2 CVS, fixed in 6.3 CVS, these come to my mind at a
glance:

GForge #343 v.clean tool=snap
RT #4247 v.build.polylines
RT #4249 v.build.polylines

Please add them to the wiki page:
http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#TODO

It seems we get the next release for 2008 only?? we need to
keep work for 6.2.3 Or more staff doing backporting...

Markus

Maciej_Sieczka2 · May 9, 2007, 4:50pm

Markus Neteler wrote:

On Wed, May 09, 2007 at 05:58:18PM +0200, Maciej Sieczka wrote:

Sorry, I don't have time to check thoroughly, but among serious bugs
remaining in 6.2.2 CVS, fixed in 6.3 CVS, these come to my mind at a
glance:

GForge #343 v.clean tool=snap
RT #4247 v.build.polylines
RT #4249 v.build.polylines

Please add them to the wiki page:
http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#TODO

Done.

It seems we get the next release for 2008 only?? we need to
keep work for 6.2.3 Or more staff doing backporting...

Would be great.

Maciek

Markus_Neteler1 · May 14, 2007, 3:23pm

On Wed, May 09, 2007 at 05:58:18PM +0200, Maciej Sieczka wrote:

Markus Neteler wrote:
> Hi,
>
> time has passed and we have fixed more problems in GRASS 6.2-CVS:
> http://grass.gdf-hannover.de/wiki/GRASS_6.2_Feature_Plan#Fixed
>
> I feel that we should get out RC1 this week to avoid further
> delay. The TODO list shows:
>
> * gis.m: any needed backports??
> -> no idea about this
> * NVIZ: volume problem (test with slovakia3d dataset) bug #4725 (Bob)
> [remove G_warning() from nviz/src/nviz_init.c if tests ok]
> http://freegis.org/cgi-bin/viewcvs.cgi/grass6/visualization/nviz/src/map_obj.c
> still not fully fixed?? nothing backported yet.
> -> I have backported this fix, it seems to cure the problem

Sorry, I don't have time to check thoroughly, but among serious bugs
remaining in 6.2.2 CVS, fixed in 6.3 CVS, these come to my mind at a
glance:

GForge #343 v.clean tool=snap
RT #4247 v.build.polylines
RT #4249 v.build.polylines

All backported (special thanks to Martin)!

Can we release RC1 now?

Markus

cmbarton · May 15, 2007, 4:17am

No objection here.

Michael

On 5/14/07 8:23 AM, "Markus Neteler" <neteler@itc.it> wrote:

All backported (special thanks to Martin)!

Can we release RC1 now?

Markus

__________________________________________
Michael Barton, Professor of Anthropology
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

phone: 480-965-6213
fax: 480-965-7671
www: http://www.public.asu.edu/~cmbarton