Hi Jukka,

Thanks, here are some more improvements :
- polygon/multipolygon case : preserve original geometry if number of geometries after cutting is still 1 - also GeometryCollection was throwing an exception. Now, it is still not cut, but it just display a (yellow) warning and let the process go on.

Michaël

Le 07/12/2015 09:08, Rahkonen Jukka (MML) a écrit :

Seems to work right with points and multipoints. It was OK already in r4578 my Michael. I suppose that the same fix corrected also another issue of previous versions: If polygons or lines were selected but the split line did not intersect with any the selected features were deleted.

Found one more small issue: If polygon is selected and the splitting line crosses some of the rings but not the whole polygon, the tool edits the source polygon by adding vertices into the locations of intersections. Perhaps the tool should check in some phase the dimensions of the intersection that is computed for polygons and revert noding if intersection is not an area?

-Jukka-

Giuseppe Aruta wrote:

Hi Jukka,

see OJ 4578. I added a small code to prevent deleting points/multipoints

Peppe

2015-12-06 23:25 GMT+01:00 Michaël Michaud <m.michael.mich...@orange.fr <mailto:m.michael.mich...@orange.fr>>:

    Hi,

    Thanks for the tests Jukka.
    Exploding selected multi-geometries even if they are not split seems
    reasonnable.
    Problem with points / multi-points should be resolved (they should be
    unmodified)
    I found another problem : the tool processed only selected features of
    the first selected layer in the layerNamePanel.
    It now processes all selected features, but the undo tool undo
    features
    from one layer at a time.

    Michaël


    Le 06/12/2015 22:38, Rahkonen Jukka (MML) a écrit :
    > Hi,
    >
    > Some comments:
    >
    > - Works well with polygon containing holes
    > - Works with multipolygons and multilinestrings
    > - Seems to explode multigeometries which may be a surprise for
    the user. On the other hand it would be very hard to define what
    else to do it multigeometries have many parts and linestring
    splits only some of them so I think that "explode and split" is OK.
    > - Tool should check that no point/multipoint geometries are
    selected. Now I get "Layer must be editable" error with such
    misuse and the selected points are deleted. Undo fortunately works.
    >
    > -Jukka Rahkonen-
    >
    > Michaël Michaud wrote:
    >
    > Hi Peppe, Jukka,
    >
    > I just improved CutFeatureTool recently added by Peppe,
    >
    > - simplified split routines
    > - can now split a feature in more than two parts
    > - more robust (should remove artefacts noticed by Jukka : can
    you confirm ?)
    > - more transactional : undo will now un-split all features
    splitted durnig an operation
    >
    > There is still a side effect which is difficult to avoid :
    > - if a linestring contains a desired self-intersection (non
    simple-linestring), splitting the linestring will node the
    self-intersection (splitting uses union op which is noding all
    input features).
    >
    > Michaël
    >
    > Le 04/12/2015 10:21, Rahkonen Jukka (MML) a écrit :
    >> Hi,
    >>
    >> The tool has good features:
    >> - can split many polygons at the same time
    >> - splits also linestrings
    >>
    >> Something to study:
    >> - Splitting a polygon with hole through the hole creates odd
    geometries. The existing Split polygon with linestring toold does
    good job with those.
    >> - Splitting multipolygons can create odd geometries.
    >>
    >> -Jukka Rahkonen-
    >>
    >>
    >>
    ----------------------------------------------------------------------
    >> -------- Go from Idea to Many App Stores Faster with Intel(R)
    XDK Give
    >> your users amazing mobile app experiences with Intel(R) XDK.
    >> Use one codebase in this all-in-one HTML5 development environment.
    >> Design, debug & build mobile apps & 2D/3D high-impact games for
    multiple OSs.
    >> http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
    >> _______________________________________________
    >> Jump-pilot-devel mailing list
    >> Jump-pilot-devel@lists.sourceforge.net
    <mailto:Jump-pilot-devel@lists.sourceforge.net>
    >> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
    >>
    >
    >
    
------------------------------------------------------------------------------
    > Go from Idea to Many App Stores Faster with Intel(R) XDK Give
    your users amazing mobile app experiences with Intel(R) XDK.
    > Use one codebase in this all-in-one HTML5 development environment.
    > Design, debug & build mobile apps & 2D/3D high-impact games for
    multiple OSs.
    > http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
    > _______________________________________________
    > Jump-pilot-devel mailing list
    > Jump-pilot-devel@lists.sourceforge.net
    <mailto:Jump-pilot-devel@lists.sourceforge.net>
    > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
    >
    >
    
------------------------------------------------------------------------------
    > Go from Idea to Many App Stores Faster with Intel(R) XDK
    > Give your users amazing mobile app experiences with Intel(R) XDK.
    > Use one codebase in this all-in-one HTML5 development environment.
    > Design, debug & build mobile apps & 2D/3D high-impact games for
    multiple OSs.
    > http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
    > _______________________________________________
    > Jump-pilot-devel mailing list
    > Jump-pilot-devel@lists.sourceforge.net
    <mailto:Jump-pilot-devel@lists.sourceforge.net>
    > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
    >


    
------------------------------------------------------------------------------
    Go from Idea to Many App Stores Faster with Intel(R) XDK
    Give your users amazing mobile app experiences with Intel(R) XDK.
    Use one codebase in this all-in-one HTML5 development environment.
    Design, debug & build mobile apps & 2D/3D high-impact games for
    multiple OSs.
    http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
    _______________________________________________
    Jump-pilot-devel mailing list
    Jump-pilot-devel@lists.sourceforge.net
    <mailto:Jump-pilot-devel@lists.sourceforge.net>
    https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel



------------------------------------------------------------------------------
Go from Idea to Many App Stores Faster with Intel(R) XDK
Give your users amazing mobile app experiences with Intel(R) XDK.
Use one codebase in this all-in-one HTML5 development environment.
Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140


_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

------------------------------------------------------------------------------
Go from Idea to Many App Stores Faster with Intel(R) XDK
Give your users amazing mobile app experiences with Intel(R) XDK.
Use one codebase in this all-in-one HTML5 development environment.
Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to