Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Be nice to others! Respect the FreeCAD code of conduct!
Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Hi,
the commit "Merge pull request #3188 from Schildkroet/grbl-post" removed the "enable spindle" and also M6 for tool-change in the resulting gcode. The spindle is moving but not turning.
OS: openSUSE Tumbleweed (KDE//usr/share/xsessions/plasma5)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20116 (Git)
Build type: Release
Branch: master
Hash: a203eeef60e4268f8be81e066252be3e0a214676
Python version: 3.8.2
Qt version: 5.14.1
Coin version: 3.1.3
OCC version: 7.4.0.beta
Locale: German/Germany (de_DE)
the commit "Merge pull request #3188 from Schildkroet/grbl-post" removed the "enable spindle" and also M6 for tool-change in the resulting gcode. The spindle is moving but not turning.
OS: openSUSE Tumbleweed (KDE//usr/share/xsessions/plasma5)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20116 (Git)
Build type: Release
Branch: master
Hash: a203eeef60e4268f8be81e066252be3e0a214676
Python version: 3.8.2
Qt version: 5.14.1
Coin version: 3.1.3
OCC version: 7.4.0.beta
Locale: German/Germany (de_DE)
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
And here comes my file for testing.
- Attachments
-
- Job_Seite_links.FCStd
- (64.99 KiB) Downloaded 34 times
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
I looked into it and it seems that these operations do not have an "active" attribute at all, so they are overlooked in the post processor.
For the moment you can go to "Mod\Path\PathScripts\post\grbl_post.py" and delete these lines (around 263ff):
I will make a new PR with an fix as soon as possible.
mfg
For the moment you can go to "Mod\Path\PathScripts\post\grbl_post.py" and delete these lines (around 263ff):
Code: Select all
if not PathUtil.opProperty(obj, 'Active'):
continue
mfg
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Thanks. I already commented these lines out as a workaround.
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Hello,
does following tool changes are output for you? I made some tests now and for me only the first M6 is output, but not the following. It seems there is something else wrong.
mfg
does following tool changes are output for you? I made some tests now and for me only the first M6 is output, but not the following. It seems there is something else wrong.
mfg
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Hi @gauna,
As I reported here https://forum.freecadweb.org/viewtopic.php?f=15&t=44665, not only the spindle commands, but some other operations are missing in the final GCode file.
It seems that your problem is the same as mine, and the pull request you report here seems to be it's origine.
Comment of the line is a workaround, but all the disabled op will also output...
@++;
Gauthier.
As I reported here https://forum.freecadweb.org/viewtopic.php?f=15&t=44665, not only the spindle commands, but some other operations are missing in the final GCode file.
It seems that your problem is the same as mine, and the pull request you report here seems to be it's origine.
Comment of the line is a workaround, but all the disabled op will also output...
@++;
Gauthier.
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
@Gaunthier
No, disabled operations are not outputted. This two lines prevent only output of comments and coolant (maybe also tool change). But the op itself is already discarded before.
- Only first M6 is output
- The preview editor does not show all headers (preview != actual file)
Was there some changes for tool change or preview recently??
OS: Windows 10 (10.0)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20310 (Git)
Build type: Release
Branch: master
Hash: 2cac4898d7d62276fbbaaef5b4cecb9bfdd29a33
Python version: 3.8.2
Qt version: 5.12.5
Coin version: 4.0.0
OCC version: 7.4.0
Locale: German/Germany (de_DE)
No, disabled operations are not outputted. This two lines prevent only output of comments and coolant (maybe also tool change). But the op itself is already discarded before.
There seems to be a general problem in output. I did some test and found out these bad behaviours:
- Only first M6 is output
- The preview editor does not show all headers (preview != actual file)
Was there some changes for tool change or preview recently??
OS: Windows 10 (10.0)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20310 (Git)
Build type: Release
Branch: master
Hash: 2cac4898d7d62276fbbaaef5b4cecb9bfdd29a33
Python version: 3.8.2
Qt version: 5.12.5
Coin version: 4.0.0
OCC version: 7.4.0
Locale: German/Germany (de_DE)
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
OK, found out, the tool change problem only occurs if you use the default tool. If you use only your created tools, it works as expected. Seems to be a problem with tool tables etc.
mfg
mfg
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Can you make a pull request?
This should fix the problem:
Code: Select all
isActive = PathUtil.opProperty(obj, 'Active')
if isActive == False:
continue
- dubstar-04
- Posts: 698
- Joined: Mon Mar 04, 2013 8:41 pm
- Location: Chester, UK
- Contact:
Re: Commit "Merge pull request #3188 from Schildkroet/grbl-post" removed "enable spindle" command in gcode
Have a look at this PR for the LinuxCNC post:
https://github.com/FreeCAD/FreeCAD/comm ... 1f227be003
Thanks,
Dan
https://github.com/FreeCAD/FreeCAD/comm ... 1f227be003
Thanks,
Dan