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 <[EMAIL PROTECTED]> wrote: > #36: v.generalize Assertion failed. > ----------------------+----------------------------------------------------- > Reporter: marisn | Owner: [email protected] > 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 > [email protected] > http://lists.osgeo.org/mailman/listinfo/grass-dev > _______________________________________________ grass-dev mailing list [email protected] http://lists.osgeo.org/mailman/listinfo/grass-dev
