A regression from #9506 where `self.multiply` was removed but not all references to it were updated.
I'm not a Python guy but from what I can see the behaviour is the same - `offsetvector` is still an `App.Vector` which `self.multiply` used to use as well.
Tested with a project that previously didn't work, and now shows a nice `Arc`-type dressup in a circular pocket.
This is a rename that got missed in #6637. I ran into this bug when trying to post a job using the `linuxcnc` post processor on a Windows 11 machine, running the weekly build titled `FreeCAD_weekly-builds-33576-2023-07-13-conda-Windows-x86_64-py310.7z`.
While working on https://github.com/FreeCAD/FreeCAD/pull/9867 I noticed my patch
showed up with a lot of linting issues in code I did not touch, to a point
where the view was very cluttered by lintian issues. Here is my second
try to reduce the number of issues discovered by the linter. Some
issues are still left, as I fail to see how to sensibly reduce the number of
parameters or local variable used.
The issue only happen when splitting jobs on tools (orderby == Tool), and when
USE_TLO was active and the preamble include G49. The first move is then done
before tool height is set, and can cause damage if the existing tool height is set
to more than the gap between the spindle and the table or work piece, when the machine
take a sudden dive straight down.
Removed move between G49 and first G43, to ensure all moves are done after G43
correctly set tool height compensation.
Rewrote code to introduce new method fixtureSetup() to ensure all orderby alternatives
behave the same way.
Fixes#9866.
While working on https://github.com/FreeCAD/FreeCAD/pull/9867 I noticed my patch
showed up with a lot of linting issues in code I did not touch, to a point
where the view was very cluttered by lintian issues. Here is my
try to reduce the number of issues discovered by the linter. Some
issues are left, as I fail to see how to sensibly reduce the number of
parameters or local variable used.
