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

Extension fails if units set to inches #19

Closed
hellocatfood opened this issue Feb 27, 2016 · 2 comments
Closed

Extension fails if units set to inches #19

hellocatfood opened this issue Feb 27, 2016 · 2 comments

Comments

@hellocatfood
Copy link

I changed the units in an inkscape document from millimetres to inches. When I send a docuemnt to the machine the plugin fails and returns the following error

Traceback (most recent call last):
File "sendto_silhouette.py", line 1042, in
e.affect()
File "/usr/share/inkscape/extensions/inkex.py", line 271, in affect
self.effect()
File "sendto_silhouette.py", line 990, in effect
mediawidth=px2mm(self.docWidth),
File "sendto_silhouette.py", line 119, in px2mm
return px*25.4/90
TypeError: unsupported operand type(s) for *: 'NoneType' and 'float'

When I change it back to millimetres it works. I think either this should be fixed of specified in the README that it only works with millimetres

@jnweiger
Copy link
Member

Ah, I haven't tried that. I usually work with px or mm. Both work fine for me. Strange that inches behave differently.

@jnweiger
Copy link
Member

Fixed in version 1.18 -- the recycled eggbot-code did not support inches. Added that.
We still miss support for units pc and pt, if someone knows the exact formula for this, please feel free to enhance the GetLengh function near

def getLength( self, name, default ):

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

No branches or pull requests

2 participants