[GRASS-dev] Minor patchesL v.transform

Hi,

I have posted some minor cosmetic / printing patches in the past to the Gforge
site, however it looks like they were either overlooked- or not deemed worth
implementing.

The patch in mind enabled printing of the transformation matrix, as computed
by v.transform, when prompted by the user. I have tried to add this
functionality with a minimal impact on module design- and followed several of
Glynn's suggested coding practices. The patches are attached to this message.

If this is too specialized for v.transform I will continue to maintain my
local copies. What do people think?

Cheers,

Dylan

--
Dylan Beaudette
Soils and Biogeochemistry Graduate Group
University of California at Davis
530.754.7341

(attachments)

transform.c.patch (1.58 KB)
v.transform.patch (2.9 KB)

Dylan Beaudette-2 wrote:

Hi,

I have posted some minor cosmetic / printing patches in the past to the
Gforge
site, however it looks like they were either overlooked- or not deemed
worth
implementing.

To me it seems to be overlooked in GForge.

I like the "trac" approach where milestones are assigned. This leads
to a roadmap which is transparent to developers and users. See
GDAL for example:
http://trac.osgeo.org/gdal/roadmap

Markus

PS: I see no reason to reject your patch.

--
View this message in context: http://www.nabble.com/Minor-patchesL-v.transform-tf4483080.html#a12786583
Sent from the Grass - Dev mailing list archive at Nabble.com.

On Wednesday 19 September 2007, Markus Neteler wrote:

Dylan Beaudette-2 wrote:
> Hi,
>
> I have posted some minor cosmetic / printing patches in the past to the
> Gforge
> site, however it looks like they were either overlooked- or not deemed
> worth
> implementing.

To me it seems to be overlooked in GForge.

I like the "trac" approach where milestones are assigned. This leads
to a roadmap which is transparent to developers and users. See
GDAL for example:
http://trac.osgeo.org/gdal/roadmap

Markus

PS: I see no reason to reject your patch.

I too like the trac approach. I am experimenting with it for some of my own
research... I will wait and see if there is anyone who can apply the patches.
Certainly not a big rush, just some functionality that I found lacking /
useful.

Cheers,

Dylan

--
Dylan Beaudette
Soils and Biogeochemistry Graduate Group
University of California at Davis
530.754.7341

On Wed, 2007-09-19 at 12:14 -0700, Dylan Beaudette wrote:

Hi,

I have posted some minor cosmetic / printing patches in the past to the Gforge
site, however it looks like they were either overlooked- or not deemed worth
implementing.

The patch in mind enabled printing of the transformation matrix, as computed
by v.transform, when prompted by the user. I have tried to add this
functionality with a minimal impact on module design- and followed several of
Glynn's suggested coding practices. The patches are attached to this message.

If this is too specialized for v.transform I will continue to maintain my
local copies. What do people think?

Committed to CVS. Thanks. lib/vector/transform looks like a good
candidate for using lib/gmath (when finished), too.

--
73, de Brad KB8UYR/6 <rez touchofmadness com>

On Wednesday 19 September 2007, Brad Douglas wrote:

On Wed, 2007-09-19 at 12:14 -0700, Dylan Beaudette wrote:
> Hi,
>
> I have posted some minor cosmetic / printing patches in the past to the
> Gforge site, however it looks like they were either overlooked- or not
> deemed worth implementing.
>
> The patch in mind enabled printing of the transformation matrix, as
> computed by v.transform, when prompted by the user. I have tried to add
> this functionality with a minimal impact on module design- and followed
> several of Glynn's suggested coding practices. The patches are attached
> to this message.
>
> If this is too specialized for v.transform I will continue to maintain my
> local copies. What do people think?

Committed to CVS. Thanks. lib/vector/transform looks like a good
candidate for using lib/gmath (when finished), too.

Thanks Brad!

I would also agree on converting lib/vector/transform to the new matrix code.

cheers,

Dylan

--
Dylan Beaudette
Soils and Biogeochemistry Graduate Group
University of California at Davis
530.754.7341