Please dont take this as a negative feedback.
I’m happy to see that there are contributer like you that get involve and act to get the Onefinity Post Processor Evolve.
That said, I did some review and wanted to give you my though and suggestion
-
The baseline used to create your revision seem to be ‘$Revision: 43008’.
This is an old revision that do not take in account Autodesk and Buildbotics changes and since onefinity post processor inherits from them this will eventually be a problem.
Autodesk last revison for onefinity is 43242 (but also need adjustment as Onefinity support asked everybody to enable UseCircularInterpolation). -
Lack of UseCircularInterpolation property definition (it’s part of last autodesk 43242)
-
New property ‘spindleDelay’ with default value of 0 second. Good idea. User can ajust if needed only.
-
New property ‘spindlePause’ with default ‘true’ value. Good idea but the default value should be ‘false’ because this is a specific need that differ from common post processor and also most VFD spindle user will not use it. User can ajust if needed only.
-
Syntax error found at line 582 (missing ; at the end of writeBlock sentence)
-
Duplicated block line 583-589 should be removed as they appear where is suppose to at line 698-704. (Edit 2021-04-21 Finally lines mandatory)
-
Need to update syntax to use the new property definitions implemented by autodesk.
-
Default value of UseCircularInterpolation set to true in order to enable arc circular movement instead of linear circular movement.
-
File in your post was not usable on my system because it has unix carriage return.
I did ajust v43242 and apply suggested change.
Here is the file (EDIT 2021-04-30) onefinity_fusion360-v43242B.cps (53.0 KB) (upload://j0eL8LLaVbdVRAKjUh0KMnNtIyQ.cps) (52.8 KB)
Note: The need to use ‘spindleDelay’ and ‘spindlePause’ is less pertinent for HY Huanyand spindle user in particular for those who set PD014 SET 1.5 – DEFINE ACCELATI0N TIME TO 1.5 SECOND (default was 10).
Thanks