Some LPE alterates some curves segments with parallel handles

Bug #1559721 reported by ivan louette
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Undecided
Alvin Penner

Bug Description

Attached is a curve segment with handles going in opposite directions along parallel axes. LPE Rotate Copies draws it erroneously, as seen if one select the object with the Node Editor (F2).

This happens for example in Bend LPE and in Rotate copies LPE

When moving around any node, or the segment itself or the entire LPE object the curvature at the right node suddently inverts.

The only way I found to remove the problem without changing my original curve is adding a node at any place on it.

Otherwise, breaking the parallelism between the two handles result in a correctly redrawn curve too.

Inkscape 0.91 devel ppa on Kubuntu 14.04

Revision history for this message
ivan louette (ivan-louette) wrote :
Revision history for this message
ivan louette (ivan-louette) wrote :

I reproduced the wrong curve systematically in any new file.

It occurs also when handles go in the same direction on parallel axes.

The parallel handles axes must not be horizontal or vertical ; any angle gives the error. It seems this is linked only to parallelism.

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Ivan the bug is related to more than one LPE, for example happens with the same path with the bend path LPE, so not is a copy rotate bug. Change the title.

summary: - LPE Rotate Copies alterates some curves segments with parallel handles
+ Some LPE alterates some curves segments with parallel handles
description: updated
Changed in inkscape:
status: New → Confirmed
Revision history for this message
Jabiertxof (jabiertxof) wrote :

SVG Test with bend Path

Revision history for this message
Jabiertxof (jabiertxof) wrote : Re: [Bug 1559721] Re: LPE Rotate Copies alterates some curves segments with parallel handles

I just update your bug. Thanks for fill it!

El dom, 20-03-2016 a las 15:18 +0000, ivan louette escribió:
> I reproduced the wrong curve systematically in any new file.
>
> It occurs also when handles go in the same direction on parallel
> axes.
>
> The parallel handles axes must not be horizontal or vertical ; any
> angle
> gives the error. It seems this is linked only to parallelism.
>

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Confirmed Debian Stretch in 0.91+devel r.14726

Revision history for this message
su_v (suv-lp) wrote :

Based on tests with archived builds using bend-curve-error.svg (r12920 ok, r12921 not ok), this issue was introduced/exposed with the changes of the 2geom update in r12921:
https://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/12921

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Thanks for test it su_v!

Revision history for this message
ivan louette (ivan-louette) wrote :

Excellent work ! And of course that was better to change the title !

jazzynico (jazzynico)
tags: added: livepatheffects
Revision history for this message
Alvin Penner (apenner) wrote :

attached is a proposed fix for this bug. I have tested it only with the file rotate-copies-curve-error.svg.
any further testing would be welcome.

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Thanks Alvin for the patch, I hope this weekend I could do some test.

Revision history for this message
ivan louette (ivan-louette) wrote : Re: [Bug 1559721] Re: Some LPE alterates some curves segments with parallel handles

Thanks for the patch ! I don't have the technical skills to test it but I am confident :-)
ivan

    Le Samedi 2 avril 2016 1h30, Alvin Penner <email address hidden> a écrit :

 attached is a proposed fix for this bug. I have tested it only with the file rotate-copies-curve-error.svg.
any further testing would be welcome.

** Patch added: "sbasis-to-bezier_1559721.diff"
  https://bugs.launchpad.net/inkscape/+bug/1559721/+attachment/4620002/+files/sbasis-to-bezier_1559721.diff

--
You received this bug notification because you are subscribed to the bug
report.
https://bugs.launchpad.net/bugs/1559721

Title:
  Some LPE alterates some curves segments with parallel handles

Status in Inkscape:
  Confirmed

Bug description:
  Attached is a curve segment with handles going in opposite directions
  along parallel axes. LPE Rotate Copies draws it erroneously, as seen
  if one select the object with the Node Editor (F2).

  This happens for example in Bend LPE and in Rotate copies LPE

  When moving around any node, or the segment itself or the entire LPE
  object the curvature at the right node suddently inverts.

  The only way I found to remove the problem without changing my
  original curve is adding a node at any place on it.

  Otherwise, breaking the parallelism between the two handles result in
  a correctly redrawn curve too.

  Inkscape 0.91 devel ppa on Kubuntu 14.04

To manage notifications about this bug go to:
https://bugs.launchpad.net/inkscape/+bug/1559721/+subscriptions

Revision history for this message
Alvin Penner (apenner) wrote :

fix committed to rev 14762.
I'll commit this as well to 2geom in a few days if there are no objections.

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Thanks Alvin, sorry, I coulden't get time to make more tests this weekend.

Revision history for this message
Alvin Penner (apenner) wrote :

same fix committed to github:
https://github.com/inkscape/lib2geom

Changed in inkscape:
status: Confirmed → Fix Committed
assignee: nobody → Alvin Penner (apenner)
Max Gaukler (mgmax)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.