Inkscape

Better interfacing to use_slow_but_correct_offset_method

Bug #770461 reported by Pablo Trabajos
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Confirmed
Wishlist
insaner

Bug Description

This bug is linked to this bug: https://bugs.launchpad.net/inkscape/+bug/550789

(NOTE by Pablo Trabajos: This was brought up on the linked question.
Please, focus this bug on creating a better interface for the use_slow_but_correct_offset_method for extensions and/or GUI.
Let's keep bug #550789 for the offset errors described there, ok?).

I'm writing Gcode tools extension (https://launchpad.net/gcodetools/) and I really need correct dynamic offset results.

I saw use_slow_but_correct_offset_method variable that defines which method will be used to calculate offset http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/annotate/head%3A/src/sp-offset.cpp#L475

How can I adjust it's value?

Could you put adjustment of use_slow_but_correct_offset_method at least into config files?

This is really needed!

See original description
Pablo Trabajos (pajarico)
Changed in inkscape:
status: New → Confirmed
Pablo Trabajos (pajarico)
description: updated
su_v (suv-lp)
Changed in inkscape:
importance: Undecided → Wishlist
tags: added: shape-editing ui
Revision history for this message
su_v (suv-lp) wrote :

The variable 'use_slow_but_correct_offset_method' has been enabled (for testing a fix for bug #384688;not as a runtime preference setting though) in 0.91+devel r14156:
http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/14156

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

See also bug report about the regressions in trunk introduced in rev 14156 by the so-called "use_slow_but_correct_offset" method (it is less correct):
* Bug #1507049 “Regression with linked / dynamic offsets (rev >= 14156)”
  https://bugs.launchpad.net/inkscape/+bug/1507049

Proposing to close this report (bug #770461) - the old method in current stable is already the more "correct" one.

insaner (insaner)
Changed in inkscape:
assignee: nobody → insaner (insaner)
Revision history for this message
insaner (insaner) wrote :

Here's a preliminary patch for review and to test for problems. I am still trying to figure out how to trigger a document refresh if the option is changed at runtime. I will submit a new patch once that's done.

Revision history for this message
insaner (insaner) wrote :

Ok, with some badly needed help from jabiertxo, I finally got it to refresh at runtime. Just click or unclick the option to have the render method for offsets change as soon as you do.

Please let me know if you find any problems, or if you object to adding this option for those of us who could benefit from not having artifacts (while we wait for a permanent fix) and the ability to switch from one render method to the other at runtime.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Related questions

Remote bug watches

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