grblgui icon indicating copy to clipboard operation
grblgui copied to clipboard

Crashing when reading NC file

Open Mgilbride opened this issue 11 years ago • 11 comments

So if I try to open a .nc file produced with the default post processor from cambam grbl gui crashes. Looking at the format of your included test files they seem to be quite different. Is grbl gui not tolerant of different file formats?

Like the visualizer. Seems robust. Looks like it is visualizing what has already been streamed to the buffer but is yet unprocessed too. Like that.

Mgilbride avatar Feb 09 '14 02:02 Mgilbride

Could you attach or send a .nc file that does not work? If the file is compatible with grbl I will fix the problem in grblgui.

cody82 avatar Feb 09 '14 02:02 cody82

This is the file I am trying. It's not a great file. Just a test file I quickly generated. It works with universal gcode sender and grbl. So I am assuming it should work with grbl gui. I do not have a clear idea of why it crashes on my system. Your gui and example files work fine on my system.

On Sat, Feb 8, 2014 at 9:48 PM, cody82 [email protected] wrote:

Could you attach or send a .nc file that does not work? If the file is compatible with grbl I will fix the problem in grblgui.

Reply to this email directly or view it on GitHubhttps://github.com/cody82/grblgui/issues/3#issuecomment-34563790 .

Mgilbride avatar Feb 09 '14 03:02 Mgilbride

I can't see the file. Can you upload it somewhere?

cody82 avatar Feb 09 '14 09:02 cody82

Sorry about that. Thought email attachment would work. Try: http://snk.to/f-chcsg5lj

Again, this is a quick test. I am not clear on if IJK are supported with the G02 & G03 commands in grbl. Am just getting my head around some of these details. The file works in Universal gcode sender's gui and visualizer. It also streams fine to a grbl loaded uno. Some commands are skipped and I need to look through to see which commands those are. This file just crashes grblgui.

Hopefully not sacrilege if I say that you and UGC could make a very nice merge.

Mgilbride avatar Feb 09 '14 20:02 Mgilbride

I see two problems with this file:

  1. There are comments in the file. This is easy to fix.
  2. G2 and G3 arc movements are not implemented in grblgui. I will see what i can do.

Until then you could try to disable arcs and comments in your CAM-software.

cody82 avatar Feb 09 '14 20:02 cody82

OK. I'll look at that too. Thanks.

On Sun, Feb 9, 2014 at 3:17 PM, cody82 [email protected] wrote:

I see two problems with this file:

  1. There are comments in the file. This is easy to fix.
  2. G2 and G3 arc movements are not implemented in grblgui. I will see what i can do.

Until then you could try to disable arcs and comments in your CAM-software.

Reply to this email directly or view it on GitHubhttps://github.com/cody82/grblgui/issues/3#issuecomment-34584869 .

Mgilbride avatar Feb 09 '14 20:02 Mgilbride

I have uploaded a new release which at least does not crash with your file. But the rendering still has some problems which I will try to fix.

cody82 avatar Feb 15 '14 00:02 cody82

Nice. I'll try it out. Thanks.

On Fri, Feb 14, 2014 at 7:14 PM, cody82 [email protected] wrote:

I have uploaded a new release which at least does not crash with your file. But the rendering still has some problems which I will try to fix.

Reply to this email directly or view it on GitHubhttps://github.com/cody82/grblgui/issues/3#issuecomment-35141196 .

Mgilbride avatar Feb 15 '14 01:02 Mgilbride

So I did try this out. I also stripped all of the comments out of the file and tried to stream it. I got a crash on streaming. Is that the rendering issues you mentioned?

Mgilbride avatar Feb 18 '14 03:02 Mgilbride

No, thats something different... I just tested the displaying of the toolpath. I could not test the streaming as I dont have an Arduino here right now. Will do that next weekend.

On Tue, Feb 18, 2014 at 4:05 AM, Mgilbride [email protected] wrote:

So I did try this out. I also stripped all of the comments out of the file and tried to stream it. I got a crash on streaming. Is that the rendering issues you mentioned?

Reply to this email directly or view it on GitHubhttps://github.com/cody82/grblgui/issues/3#issuecomment-35348432 .

cody82 avatar Feb 18 '14 07:02 cody82

The first line "G21 G90 G64 G40" seems to be a problem for grbl. It just does nothing there and wont continue. I removed it and streaming works then. I'm testing with the newest grbl from git.

cody82 avatar Mar 14 '14 22:03 cody82