FluffyMortain / heekscnc

Automatically exported from code.google.com/p/heekscnc
Other
0 stars 0 forks source link

Profiling units passed to python are scaled incorrectly for the offset_extra parameter #235

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Using the 'Program' entry in the design tree, choose to use inches as units
2. Create a closed sketch in the XY plane.  (I select a face from an imported 
solid and then do 'convert to sketch' to do this)
3. Select it in the design tree, and then do a profiling machine operation from 
it.
4. Enter a non-zero value for offset_extra in the profiling properties list
5. Generate Gcode with the G0 icon up on the top toolbar

What is the expected output? What do you see instead?

6. Notice that the python code generated has a different offset_extra value 
than was entered in the properties list.  They should match, I think.
Note that entering '1' in the properties list yields offset_extra = 25.4 in the 
python code.  It seems to be doing an inappropriate unit conversion from in to 
mm somehow.

What version of the product are you using? On what operating system?
HeeksCNC 0.13.1
Windows 7 Pro

Please provide any additional information below.
The attached file shows how I have to put the wrong value in the offset_extra 
in order to get my desired 0.015 inch clearance.

Original issue reported on code.google.com by kurt.zoo...@gmail.com on 3 Aug 2010 at 5:41

Attachments:

GoogleCodeExporter commented 9 years ago
This was fixed, see issue 218.
I have now made a new download HeeksCNC 0.14.0 with this fix in.

Original comment by danhe...@gmail.com on 4 Aug 2010 at 7:38

GoogleCodeExporter commented 9 years ago
Oops!  Please accept my apology for filing a duplicate.  Next time I'll filter 
on all issues instead of just open issues when looking to see if something has 
already been filed.  Thanks again to all the developers for this fantastic 
application!

Original comment by kurt.zoo...@gmail.com on 4 Aug 2010 at 3:04