occasional bad behaviour of envelope deformation LPE (rev >= 12921)

Bug #1428683 reported by Alvin Penner on 2015-03-05
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Medium
Alvin Penner

Bug Description

- Windows XP, Inkscape 0.91+devel r13954 (Feb 28 2015)
- load the file envelope_deform.svg which contains a path with 3 nodes and an Envelope Deformation LPE
- note that it contains some apparent discontinuities or jagged segments
- select the node tool F2, hover over the path until you see the red path flicker temporarily.
- click on the path and select the left-most node which is off the page at (-84,214)
- slowly drag the left-most node up and down, to see that most of the response to this dragging is very smooth and predictable except in this one very localized area where it is pathological.

- reproduced on Inkscape 0.91

Alvin Penner (apenner) wrote :
su_v (suv-lp) on 2015-03-05
tags: added: livepatheffects
jazzynico (jazzynico) wrote :

Reproduced on Windows XP, Inkscape 0.91 and trunk revision 13979.
Not reproduced with 0.48.5.

Changed in inkscape:
milestone: none → 0.91.1
status: New → Triaged
tags: added: regression
su_v (suv-lp) wrote :

Based on tests with archived builds on OS X 10.7.5:
- not reproduced with rev <= 12920,
- reproduced with rev >= 12921;
this regression seems related to the update of 2geom in r12921
http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/12921

Earlier reports affected by the same revision:
Bug #1272119 “trunk: poor rendering of path effects after 2geom update (rev >= 12921)” (fixed)
Bug #1312333 “trunk: incorrect rendering of circle segment (rev >= 12921)” (fixed)

tags: added: 2geom
summary: - occasional bad behaviour of envelope deformation LPE
+ occasional bad behaviour of envelope deformation LPE (rev >= 12921)
su_v (suv-lp) on 2015-03-09
Changed in inkscape:
importance: Undecided → Medium
Alvin Penner (apenner) wrote :

fix committed to rev 14005.

The fix involves the 2geom function sbasis_to_cubic_bezier, so it will have to be committed separately to 2geom as well. I thought I'd commit it here first to allow testing and then commit it to 2geom after a few days or so.

Changed in inkscape:
status: Triaged → Fix Committed
assignee: nobody → Alvin Penner (apenner)
su_v (suv-lp) on 2015-03-14
Changed in inkscape:
milestone: 0.91.1 → 0.92
tags: added: backport-proposed
Alvin Penner (apenner) wrote :

fix committed separately to lib2geom rev 2286

ScislaC (scislac) wrote :

trunk r14005 backported in 0.91.x r13752

Changed in inkscape:
milestone: 0.92 → 0.91.1
tags: removed: backport-proposed
jazzynico (jazzynico) on 2017-01-28
Changed in inkscape:
milestone: 0.91.1 → 0.92
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers