Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inkscape to Uniconvertor: .plt file output scale discrepancy - UC/Sk1 still inputs at 90dpi? #44

Closed
8thrule opened this issue Jan 23, 2017 · 5 comments

Comments

@8thrule
Copy link

8thrule commented Jan 23, 2017

Greetings,

Thank you for the Uniconvertor patches for Inkscape. The recent change in Inkscape to 96dpi may combine with code in Uniconvertor/SK1 to result in output scale issues:

Events:
Successful installation of Inkscape patch files for Uniconvertor.
Successful output of .plt file from Inkscape via Uniconvertor.
However, scale of .plt file is larger than original... 1:1.06667.

Using:
Win 8.1, 64bit,
Inkscape 0.92.0 r15299 (now has default scale of 96dpi)
Uniconvertor 1.1.5 (most recent, AFAICT)

More info:
https://bugs.launchpad.net/inkscape/+bug/1658260
Comment number 18 +

Any insights are greatly appreciated.

Regards,
TD
tylerdurden (8thrule)

plotters.zip

@raboof
Copy link
Contributor

raboof commented Dec 22, 2017

With the latest nightly (2.0rc3) converting inkscape svg to plt and then back into Inkscape again also produces a (in my case much) smaller object.

@sk1project
Copy link
Owner

sk1project commented Jan 27, 2019

@8thrule @raboof we have added optional parameter for svg processing: --svg_dpi
Default value is 90, but you can use 96 to resolve issue.

uniconvertor file.svg file.plt --svg_dpi=96

Hope this fixes the issue. You may test changes using uniconvertor night builds:
https://sk1project.net/viewpage.php?page_id=26

@jnweiger
Copy link

@sk1project Please note that the dpi default is 96, as per SVG standard. The standard once mentioned 90, but was changed to 96 a few years ago. SVG derives its defaults from the CSS definition https://www.w3.org/TR/CSS21/syndata.html#length-units

@raboof when things are 'much smaller' then you probably ran into a 72dpi vs 96dpi snakepit. Illustrator is the most prominent advocat of 72dpi as a default (due to their very own understanding of standard).

@sk1project
Copy link
Owner

@jnweiger unfortunately we need correctly processing existing SVG files but not to be Inkscape compatible only. Nevertheless we reopen this issue to find better solution than implemented.

@sk1project sk1project reopened this Jan 29, 2019
@sk1project
Copy link
Owner

@jnweiger @8thrule @raboof we have refactored code related to dpi issue. Now we have dpi auto detection. Nevertheless you may force dpi using --svg_dpi=... option. Also to have exact translation result just don't use px as a document unit. Set mm, in, cm etc. and translation will be accurate. This is naturally because cutting plotter doesn't operate with pixels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants