search

cody82 / grblgui

GUI for grbl.
http://youtu.be/kqCFuzl11s4
51 stars 19 forks source link

Crashing when reading NC file #3

Open Mgilbride opened 10 years ago

Mgilbride commented 10 years ago

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.

cody82 commented 10 years ago

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.

Mgilbride commented 10 years ago

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 notifications@github.com 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 .

cody82 commented 10 years ago

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

Mgilbride commented 10 years ago

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.

cody82 commented 10 years ago

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.

Mgilbride commented 10 years ago

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

On Sun, Feb 9, 2014 at 3:17 PM, cody82 notifications@github.com 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 .

cody82 commented 10 years ago

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.

Mgilbride commented 10 years ago

Nice. I'll try it out. Thanks.

On Fri, Feb 14, 2014 at 7:14 PM, cody82 notifications@github.com 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 commented 10 years ago

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?

cody82 commented 10 years ago

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 notifications@github.com 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 commented 10 years ago

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.