Object to path, refocus - extra nodes generated

Bug #257289 reported by hash
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Undecided
Johan Engelen

Bug Description

When you do object to path with circle there is 1 node to many, and when you move one of the overlapping nodes it is duplicated when path looses focus.
win xp Inkscape19635-0808111635 build.

Tags: 2geom
Revision history for this message
hash (hash-g) wrote :
Revision history for this message
prkos (prkos) wrote :

I can confirm 1 node too many in Ellipse, Rectangle and Star tool, on fedora 9, Inkscape build 19587.

I can't confirm the simplify path part. Do you have any reason to believe those two problems are connected? If not please report it in another bug and give a more detailed steps to reproduce.
I created a path with a pencil tool, simplified it, there was one point where 2 nodes were overlapping but I couldn't get 3 nodes there after refocus.

Changed in inkscape:
status: New → Incomplete
Revision history for this message
hash (hash-g) wrote :

I thought that because the nodes are overlapping the cases are related to each other as in something is converted to path by an algorithm and that algorithm places 2 nodes in the same place.
See the video(xvid/no-audio) how you can reproduce.

hash (hash-g)
Changed in inkscape:
status: Incomplete → Invalid
Revision history for this message
prkos (prkos) wrote :

I can confirm the second part of the report in build 19587, can't test in a more recent build cos of the move node crash.

It doesn't seem to be related to Simplify path or changing node modes (as in hash's video), I guess in the end those two symptoms might be related. The simplest way to reproduce:

1. create ellipse
2. convert to path - it will create 5 nodes, the first and last overlap
3. move one of the 2 overlapping nodes using Node tool, I think the top one is always the first
4. click on empty canvas twice to remove focus from the path
5. click on the path again to focus it
6. the moved node is duplicated (there are now 2 nodes there overlapping)
7. it can be repeated always moving one of the overlapping nodes, it always adds one more node to the path

Revision history for this message
prkos (prkos) wrote :

Possibly related to closing paths in 2geom.

description: updated
Changed in inkscape:
status: Invalid → Confirmed
Revision history for this message
Johan Engelen (johanengelen) wrote :

This is related to closing paths in SVG.
For straight line paths, one can specify 3 lines and say it is closed to create a quadrilateral. For curved paths, this is not possible, one has to explicitly mention the closing line segment. Then still, the path is marked as closed, which means that would be a zero length closing line segment. Inkscape tries to "see" this zero-length segment. I think sometimes it fails due to floating point rounding errors.
So now I am trying to fix it by being a bit more relaxed about floating point rounding errors.

Unfortunately, I cannot check whether this is fixed now or not. please try svn rev. 19841

Changed in inkscape:
assignee: nobody → johanengelen
status: Confirmed → Incomplete
Revision history for this message
worms_x (wormsxulla) wrote :

I can confirm this bug on build Inkscape 0.46+devel, built Sep 8 2008, (windows 2000 sp4)

http://inkscape.modevia.com/win32/Inkscape19816-0809080916.7z

I draw an ellipse, do "object to path", and I'm getting a path with 5 nodes instead of 4 (written in the bottom bar)

But when i switch to the node tool and select the top node, then the bottom bar says : 1 of 4 nodes selected

I can't reproduce the steps 4 to 7 to get duplicated nodes

Revision history for this message
John Cliff (johncliff) wrote :

Cant reproduce with current SVN build. Was getting it a lot the other day tho, so I'm voting its fixed.

Revision history for this message
Johan Engelen (johanengelen) wrote :

Thanks for mentioning the node count in the status bar. That node count is a bit "dumb" so I'll fix it!

Revision history for this message
Johan Engelen (johanengelen) wrote :

Fix the nodecounting in the statusbar now as well.

Revision history for this message
ScislaC (scislac) wrote :

Everything looks like it used to. Thanks as always Johan!

Changed in inkscape:
status: Incomplete → 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.