[GRASS-dev] Re: [GRASS GIS] #36: v.generalize Assertion failed.

[Daniel Bundala]

Yes, it should use GRASS_EPSILON. Can someone please change it? And as
far as I remember, there is one more hard-coded constant just a few
lines belows.
Thanks

Daniel

On Feb 10, 2008 9:10 AM, GRASS GIS <trac at osgeo.org> wrote:
> #36: v.generalize Assertion failed.
> ----------------------+-----------------------------------------------------
>   Reporter:  marisn   |       Owner:  grass-dev at lists.osgeo.org
>       Type:  defect   |      Status:  new
>   Priority:  major    |   Milestone:  6.4.0
>  Component:  default  |     Version:  svn-trunk
> Resolution:           |    Keywords:
> ----------------------+-----------------------------------------------------
> Comment (by hamish):
>
>  Daniel Bundala wrote on grass-dev:
>  > Hello,
>  >
>  > Finally, I had some time to look at it carefully. It seems to me that
>  > the problem is the same as the one I mentioned earlier. I made a patch
>  > some time ago, but for some obscure reasons, it did not propagate to
>  > the SVN. I think that the patch should fix it as it is the only
>  > difference between the version in SVN and on my computer.
>  >
>  > Daniel
>  >
>  > PS: I am not a big expert on CVS/SVN so the patch may be little out of
>  > date.....
>
>
>  The patch applied fine, now in SVN/trunk (r30050) and 6.3 release branch
>  (r30051). It still needs testing.
>
>
>  Should it use GRASS_EPSILON from include/gis.h instead of hardcoded 1e-12
>  ?
>
>
>  just do:
>  {{{
>  $ svn up vector/v.generalize
>  # make changes
>  $ vi vector/v.generalize/smoothing.c
>  # check it
>  $ svn diff vector/v.generalize
>  $ svn diff vector/v.generalize > v.gen_fix.diff
>  }}}
>
>  "svn up" after changes should try and merge in any new updates which don't
>  directly conflict.
>
>
>  Hamish
>
> --
> Ticket URL: <http://trac.osgeo.org/grass/ticket/36#comment:5>
>
> GRASS GIS <http://grass.osgeo.org>
> GRASS Geographic Information System (GRASS GIS) - http://grass.osgeo.org/
> _______________________________________________
> grass-dev mailing list
> grass-dev at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/grass-dev
>