[GRASS-dev] Re: [GRASS GIS] #105: r.in.xyz: zscale parameter
addition patch
GRASS GIS
trac at osgeo.org
Sat Mar 29 00:11:11 EDT 2008
#105: r.in.xyz: zscale parameter addition patch
--------------------------+-------------------------------------------------
Reporter: neteler | Owner: hamish
Type: enhancement | Status: assigned
Priority: major | Milestone: 6.4.0
Component: default | Version: svn-trunk
Resolution: | Keywords:
--------------------------+-------------------------------------------------
Changes (by hamish):
* status: new => assigned
* owner: grass-dev at lists.osgeo.org => hamish
* cc: grass-dev at lists.osgeo.org (added)
Comment:
Hi,
Sorry I missed the earlier discussion of the -i flag [for ignoring
malformed lines in the data stream]; I had been out of town.
In general my plan for r.in.xyz was to keep the interface (and code) as
simple as possible. The idea is to keep the methodology clean + light, and
so the code bug free + fast.
The code for the -i flag won't slow down the module any (vs prior) and
doesn't really add program complexity, so I don't have a technical problem
with it. Still, for reducing methodology mistakes & encouraging a hands-
on approach to data management, I consider it preferable that the user
should be forced to do something to fix their broken input files rather
than hack their way around them without careful study of what the problem
really is. If there are many files to process they could filter the stream
through any number of UNIX tools like sed or awk and then pipe the result
to r.in.xyz's stdin in some automated way.
--
A zscale= option seems like a reasonable request. Is the purpose that the
elevation data comes in feet and you want it in meters?
I will review the patch, run some speed tests, and take care of committing
as soon as time allows.
Hamish
--
Ticket URL: <http://trac.osgeo.org/grass/ticket/105#comment:1>
GRASS GIS <http://grass.osgeo.org>
GRASS Geographic Information System (GRASS GIS) - http://grass.osgeo.org/
More information about the grass-dev
mailing list