d.rast.leg: invalid literal for float(): rectangle

[pertusus]

I get a traceback for d.rast.leg.

d.rast.leg raster_selection_basins_change lines=2

Traceback (most recent call last):
  File "/usr/local/grass-7.0.0svn//scripts/d.rast.leg", line 167, in <module>
    main()
  File "/usr/local/grass-7.0.0svn//scripts/d.rast.leg", line 103, in main
    f = tuple([float(x) for x in s.split()[1:5]])
ValueError: invalid literal for float(): rectangle:

This is triggered by a wrong parsing of d.info which returns:

frame rectangle: 0.000000 640.000000 0.000000 480.000000

Original code is

[float(x) for x in s.split()[1:5]]

which naturally the to x being 'rectangle:'

I attach a patch where I propose instead:

[float(x) for x in s.split(":")[1].split()[0:4]]

It seems more robust, though I don't know if the : will be there forever. Overall, the approach is not very robust, but I have no idea on what's going on so I only propose that fix which may not be appropriate in other cases.

[pertusus]

patch to split first d.info output at :

[neteler]

Ticket retargeted after 7.0.1 milestone closed

[neteler]

Ticket retargeted after milestone closed

[neteler]

I checked with the NC dataset, this one works:

GRASS 7.0.3svn (nc_spm_08):~ > g.region raster=basin_50K
GRASS 7.0.3svn (nc_spm_08):~ > d.mon wx0
GRASS 7.0.3svn (nc_spm_08):~ > d.rast.leg basin_50K lines=2
 100%
 100%
GRASS 7.0.3svn (nc_spm_08):~ > d.rast.leg basin_50K 
 100%
 100%

Do you have an North Carolina example which fails?

[neteler]

Ticket retargeted after milestone closed

[neteler]

Ticket retargeted after 7.0.3 milestone closed

[neteler]

Is the proposed patch still needed?

[martinl]

No activity last 8 months, closing, feel free to reopen if needed.