[Gvsig_english] gvSIG 1.9 patch for managing measured 3D Polylines of Shapefiles

Benjamin Ducke benjamin.ducke at oxfordarch.co.uk
Mon Mar 8 19:49:02 CET 2010


A diff file contains a list of lines to match
in the target file, a list of lines to delete
if the match is found and a list of lines to
add. Sometimes, the two files can differ in minor
respects, e.g. if some extra blank lines were
inserted or some comments or new line breaks.
In that case, the lines cannot be exactly matched
and a bit of tolerance (or fuzziness) needs to
be allowed to find the next best match.
That's what happened in your case.

Cheers,

Ben

----- Original Message -----
From: "Francisco José Peñarrubia" <fpenarru at gmail.com>
To: "Users and Developers mailing list" <gvsig_internacional at listserv.gva.es>
Sent: Monday, March 8, 2010 7:43:20 PM GMT +01:00 Amsterdam / Berlin / Bern / Rome / Stockholm / Vienna
Subject: Re: [Gvsig_english] gvSIG 1.9 patch for managing measured 3D Polylines of Shapefiles

@ Benjamin. I don't use command line for this. I mainly use windows, and 
console svn is unknown for me.
@ Flavio. It's ok know. I finally managed to put this changes inside 
SVN. I used a button "Guess", wich states Maximum fuzz factor. I don't 
know what it means, but after pressing it, it got "52" and the changes 
appeared in this dialog.

Extrange, isn't it??.

Cheers, and once again, thanks for the patch.

Fran.

Flavio Pompermaier escribió:
> I did the patch via Subversive plugin of Eclipse, as I did the first 
> time.
> If there are problems with the last I sent I could remake it tomorrow.
> I'm not that good at reading diffs.. :(
>
> 2010/3/8 Benjamin Ducke <benjamin.ducke at oxfordarch.co.uk 
> <mailto:benjamin.ducke at oxfordarch.co.uk>>
>
>     Oh, yes, I applied the patch manually because I wanted to
>     understand what it does. Didn't even realize that Eclipse
>     does patches. Can it book flights and boil eggs too, now?
>
>     Does the "patch" command line tool not work for you?
>
>     Cheers,
>
>     Ben
>
>     ----- Original Message -----
>     From: "Francisco José Peñarrubia" <fpenarru at gmail.com
>     <mailto:fpenarru at gmail.com>>
>     To: "Users and Developers mailing list"
>     <gvsig_internacional at listserv.gva.es
>     <mailto:gvsig_internacional at listserv.gva.es>>
>     Sent: Monday, March 8, 2010 6:54:32 PM GMT +01:00 Amsterdam /
>     Berlin / Bern / Rome / Stockholm / Vienna
>     Subject: Re: [Gvsig_english] gvSIG 1.9 patch for managing measured
>     3D Polylines of Shapefiles
>
>     You are right, the diff file contains this lines..... But when I
>     try to
>     patch libFMap, Eclipse does nothing...
>
>     I will investigate further....
>
>     Fran.
>
>     Benjamin Ducke escribió:
>     > Hmmm, the patch I saved from Flavio's last email has
>     > this, following line 515:
>     >
>     > -                     case (SHP.POLYLINE2D):
>     > -                     case (SHP.POLYLINE3D):
>     > +                     case SHP.POLYLINE2D:
>     >
>     >                               auxType = auxType | FShape.LINE;
>     >                               break;
>     > -
>     > -                     case (SHP.POLYLINEM):
>     > +                     case SHP.POLYLINEM:
>     >
>     >                               auxType = auxType | FShape.LINE |
>     FShape.M;
>     >                               break;
>     > +                     case SHP.POLYLINE3D:
>     >
>     > +                             auxType = auxType | FShape.LINE |
>     FShape.Z;
>     >
>     > +                             break;
>     >
>     >
>     > ... is that not it?
>     >
>     > Flavio, I think we need your opinion!
>     >
>     > Ben
>     >
>     > ----- Original Message -----
>     > From: "Francisco José Peñarrubia" <fpenarru at gmail.com
>     <mailto:fpenarru at gmail.com>>
>     > To: "Users and Developers mailing list"
>     <gvsig_internacional at listserv.gva.es
>     <mailto:gvsig_internacional at listserv.gva.es>>
>     > Sent: Monday, March 8, 2010 6:15:25 PM GMT +01:00 Amsterdam /
>     Berlin / Bern / Rome / Stockholm / Vienna
>     > Subject: Re: [Gvsig_english] gvSIG 1.9 patch for managing
>     measured 3D Polylines of Shapefiles
>     >
>     > Hi Ben.
>     >
>     > Sorry, I don't understand. Why do you feel irritated??.
>     >
>     > Flavio sent some months ago a patch. I put the patch in gvSIG
>     official
>     > SVN, and now, comparing this new patch and the code
>     >
>     >           case SHP.POLYLINE2D:
>     >                 auxType = auxType | FShape.LINE;
>     >                 break;
>     >             case SHP.POLYLINEM:
>     >                 auxType = auxType | FShape.LINE | FShape.M;
>     >                 break;
>     >             case SHP.POLYLINE3D:
>     >                 auxType = auxType | FShape.LINE | FShape.Z;
>     >                 break;
>     >
>     >
>     > I can't find this code inside the last patch. It seems (for me,
>     at last)
>     > that the patch maybe and old one, and that's why I'm asking his
>     opinion...
>     >
>     > Cheers.
>     >
>     > Fran.
>     >
>     > Benjamin Ducke escribió:
>     >
>     >> Hi Fran,
>     >>
>     >> this irritates me a little. I just merged this patch
>     >> into the OADE codebase and everything was in there,
>     >> just as Flavio said it should.
>     >>
>     >> Cheers,
>     >>
>     >> Ben
>     >>
>     >> ----- Original Message -----
>     >> From: "Francisco José Peñarrubia" <fpenarru at gmail.com
>     <mailto:fpenarru at gmail.com>>
>     >> To: "Users and Developers mailing list"
>     <gvsig_internacional at listserv.gva.es
>     <mailto:gvsig_internacional at listserv.gva.es>>
>     >> Sent: Monday, March 8, 2010 5:30:21 PM GMT +01:00 Amsterdam /
>     Berlin / Bern / Rome / Stockholm / Vienna
>     >> Subject: Re: [Gvsig_english] gvSIG 1.9 patch for managing
>     measured 3D Polylines of Shapefiles
>     >>
>     >> Hi Flavio.
>     >>
>     >> First of all, thanks for the patch.
>     >>
>     >> And now, my question: It seems the patch file doesn't have this
>     >> changes... Is it possible that you sent and old patch? (The old
>     patch is
>     >> already in SVN).
>     >>
>     >> Best regards.
>     >>
>     >> Fran.
>     >>
>     >> Flavio Pompermaier escribió:
>     >>
>     >>
>     >>> Hi Francisco,
>     >>> I discovered these days that the patch for managing 4d
>     shapefiles I
>     >>> sent to you some weeks ago is uncomplete.
>     >>> 4D shapefiles are read correctly with that patch, but if I start
>     >>> editing them and then I save them, the shapeWriter do not work as
>     >>> expected.
>     >>> The malfunctioning is caused by the fact that when the shapeWriter
>     >>> tries to get the layer type
>     >>> the IndexedShpDriver collapse 2d and 4d shapefiles to the same
>     type of
>     >>> layer:
>     >>>
>     >>>             case (SHP.POLYLINEM):
>     >>>                 auxType = auxType | FShape.LINE | FShape.M;
>     >>>                 break;
>     >>>             case (SHP.POLYLINE2D):
>     >>>             case (SHP.POLYLINE3D):
>     >>>                 auxType = auxType | FShape.LINE;
>     >>>                 break;
>     >>>
>     >>> while it must be:
>     >>>
>     >>>            case SHP.POLYLINE2D:
>     >>>                 auxType = auxType | FShape.LINE;
>     >>>                 break;
>     >>>             case SHP.POLYLINEM:
>     >>>                 auxType = auxType | FShape.LINE | FShape.M;
>     >>>                 break;
>     >>>             case SHP.POLYLINE3D:
>     >>>                 auxType = auxType | FShape.LINE | FShape.Z;
>     >>>                 break;
>     >>>
>     >>> The new version of the patch of libFMap is attached to the mail.
>     >>> Best regards,
>     >>> Flavio
>     >>>
>     >>>
>     >>> 2010/2/1 Flavio Pompermaier <fla83tn at libero.it
>     <mailto:fla83tn at libero.it> <mailto:fla83tn at libero.it
>     <mailto:fla83tn at libero.it>>>
>     >>>
>     >>>     Hi Francisco,
>     >>>     I was thinking nobody read my email..
>     >>>     However, I tested the solution against the attached shapefile,
>     >>>     which has x,y,z and m dimension (although z is always 0...).
>     >>>     I didn't test legends and labelling because by now I use
>     it like a
>     >>>     normal 3D shapefile.
>     >>>     But if you do it it should be certainly a good idea!
>     >>>     Moreover I didn't test non-indexed shapefiles but I don't even
>     >>>     know if they exist..and also if they do
>     >>>     it seems to me that the code to manage them is always passing
>     >>>     through the classes I modified.
>     >>>
>     >>>     Thanks for keeping me up to date,
>     >>>     Flavio
>     >>>
>     >>>
>     >>>     2010/1/29 Francisco José Peñarrubia <fpenarru at gmail.com
>     <mailto:fpenarru at gmail.com>
>     >>>     <mailto:fpenarru at gmail.com <mailto:fpenarru at gmail.com>>>
>     >>>
>     >>>         Hi Flavio.
>     >>>
>     >>>         Sorry about this late response... :-(.
>     >>>
>     >>>         About the patch, we are going to evaluate it. Please,
>     can you
>     >>>         send us
>     >>>         some shapefiles to test, and one or two use cases?.
>     >>>         We are going to test also some legends, and labelling
>     (just in
>     >>>         case you
>     >>>         haven't tested it yet).
>     >>>
>     >>>         Thanks a lot for this collaboration, and keep going,
>     please.
>     >>>
>     >>>         Fran Peñarrubia.
>     >>>         gvSIG Team
>     >>>
>     >>>         Flavio Pompermaier escribió:
>     >>>         > Hi to all,
>     >>>         > In the project I' working on I had the need of
>     managing M
>     >>>         dimension of
>     >>>         > PolylineZ contained in shapefiles
>     >>>         > and I had to modify the libFMap and read also M
>     coordinate for
>     >>>         > Shape.POLYLINE3D.
>     >>>         > The patch is attached to this mail.
>     >>>         > I'd like to know whether this patch could be added of
>     >>>         interest for a
>     >>>         > cumulative further patch of gvSIG 1.9.
>     >>>         >
>     >>>         >
>     >>>        
>     -------------------------------------------------------------------------------------------------------------------------------------------------------
>     >>>         >  Brief summary:
>     >>>         >
>     >>>        
>     -------------------------------------------------------------------------------------------------------------------------------------------------------
>     >>>         > - Class ShapeZMFactory proxies the creation of
>     Z-geometries
>     >>>         with four
>     >>>         > coordinates. By now only polylines
>     >>>         > are managed but, if somedays there will be the need of
>     >>>         managing also
>     >>>         > Zpoints, Zmultipoints and Zpolygons
>     >>>         > with four dimension, you can always implement method
>     for their
>     >>>         > creation in the aforementioned class.
>     >>>         >
>     >>>         > - Class FPolyline3DM extends FPolyline3D in order to
>     manage m
>     >>>         > dimension of PolylineZ shapes
>     >>>         >
>     >>>         > - Class IndexedShpDriver instead manage the reading of m
>     >>>         coordinate
>     >>>         > from shapefiles..
>     >>>         >
>     >>>        
>     --------------------------------------------------------------------------------------------------------------------------------------------------------
>     >>>         >
>     >>>         > If anyone knows if I've forgotten to touch other
>     classes in
>     >>>         order to
>     >>>         > make my patch complete is pleased to let
>     >>>         > me know about it.
>     >>>         >
>     >>>         > Best regards,
>     >>>         > Flavio Pompermaier
>     >>>         >
>     >>>         >
>     >>>        
>     ------------------------------------------------------------------------
>     >>>         >
>     >>>         > _______________________________________________
>     >>>         > Gvsig_internacional mailing list
>     >>>         > Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     >>>         <mailto:Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>>
>     >>>         >
>     >>>        
>     http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >>>         >
>     >>>
>     >>>         _______________________________________________
>     >>>         Gvsig_internacional mailing list
>     >>>         Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     >>>         <mailto:Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>>
>     >>>        
>     http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >>>
>     >>>
>     >>>
>     >>>
>     ------------------------------------------------------------------------
>     >>>
>     >>> _______________________________________________
>     >>> Gvsig_internacional mailing list
>     >>> Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     >>>
>     http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >>>
>     >>>
>     >> _______________________________________________
>     >> Gvsig_internacional mailing list
>     >> Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     >> http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >>
>     >>
>     >>
>     >> ------
>     >> Files attached to this email may be in ISO 26300 format (OASIS
>     Open Document Format). If you have difficulty opening them, please
>     visit http://iso26300.info for more information.
>     >>
>     >> _______________________________________________
>     >> Gvsig_internacional mailing list
>     >> Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     >> http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >>
>     >>
>     >
>     > _______________________________________________
>     > Gvsig_internacional mailing list
>     > Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     > http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >
>     >
>     > ------
>     > Files attached to this email may be in ISO 26300 format (OASIS
>     Open Document Format). If you have difficulty opening them, please
>     visit http://iso26300.info for more information.
>     >
>     > _______________________________________________
>     > Gvsig_internacional mailing list
>     > Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     > http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>     >
>
>     _______________________________________________
>     Gvsig_internacional mailing list
>     Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>
>
>     ------
>     Files attached to this email may be in ISO 26300 format (OASIS
>     Open Document Format). If you have difficulty opening them, please
>     visit http://iso26300.info for more information.
>
>     _______________________________________________
>     Gvsig_internacional mailing list
>     Gvsig_internacional at listserv.gva.es
>     <mailto:Gvsig_internacional at listserv.gva.es>
>     http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Gvsig_internacional mailing list
> Gvsig_internacional at listserv.gva.es
> http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional
>   

_______________________________________________
Gvsig_internacional mailing list
Gvsig_internacional at listserv.gva.es
http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_internacional



------
Files attached to this email may be in ISO 26300 format (OASIS Open Document Format). If you have difficulty opening them, please visit http://iso26300.info for more information.



More information about the Gvsig_internacional mailing list