gerritv / Grbl-Panel

A control panel for Grbl
MIT License
180 stars 102 forks source link

GRBL Panel crashes when soft limit is enabled #108

Closed alfredabounasr closed 7 years ago

alfredabounasr commented 7 years ago

when soft limit is enabled and when the value of any axis reaches the soft limit value, GRBL Panel crashes

gerritv commented 7 years ago

I can't duplicate your error in 1.0.9.13 using soft limits, I get ALARM:2 in the status area with Status set to ALARM.

You will need to provide a few more hints, e.g. what version of Grbl, what version of GrblPanel, what are your limits set to, do you have Hard limits enabled,

Can you screen shot the Status area after enabling Verbose, so I can see what Grbl is sending back.

alfredabounasr commented 7 years ago

Hi, I'm using the latest grbl on arduino with the latest grbl-panel software

alfredabounasr commented 7 years ago

I'm using the latest grbl on arduino and the latest grbl-panel. Hard limit is enabled The limits are set to 200mm for testing purposes and all are positive numbers.

gerritv commented 7 years ago

Is it 1.1f or 0.9 Grbl? I tested just now with both 1.1 and 0.9 and soft limits work with no crash. You need to provide me with the exact Grbl version and a screen shot of the Status area with Verbose turned on. Without information I can't help.

alfredabounasr commented 7 years ago

It's grbl 1.1f

On Thu, Apr 27, 2017 at 2:20 PM Gerrit Visser notifications@github.com wrote:

Is it 1.1f or 0.9 Grbl?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gerritv/Grbl-Panel/issues/108#issuecomment-297686186, or mute the thread https://github.com/notifications/unsubscribe-auth/AamsdUa5GhobwkpCC7e_-SkiN2rxPb_Jks5r0Hn4gaJpZM4NFJVQ .

gerritv commented 7 years ago

I am not going to play 20 questions any longer. Do a think about what you are providing, can you fix it with that info? Read my previous requests, no info, no fix as I can't reproduce it with the set up that I have.

alfredabounasr commented 7 years ago

Hello,

here are the hints that you asked for:

GRBL version : 1.1f.20170131 Grbl-Panel Version : 1.0.9.13 Limits set to 200mm Hard Limits is enabled

Thank you and sorry for the inconvenient.

gerritv commented 7 years ago

Yes, but I also asked for a screenshot of the Status section with Verbose turned on. Since I can't duplicate the problem I need to see what Grbl is sending that makes GrblPanel crash.

alfredabounasr commented 7 years ago

The screenshot is attached within the last email i sent. Grbl is returning "error:15"

On Fri, Apr 28, 2017 at 3:37 PM Gerrit Visser notifications@github.com wrote:

Reopened #108 https://github.com/gerritv/Grbl-Panel/issues/108.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gerritv/Grbl-Panel/issues/108#event-1062049453, or mute the thread https://github.com/notifications/unsubscribe-auth/Aamsdecx3N9bzuFFf-0OTxZxYjSiUAWVks5r0d1-gaJpZM4NFJVQ .

gerritv commented 7 years ago

Unfortunately images don't come through when responding by email :-( Knowing it is error:15 now I know it is Jog command that it fails on.

I know what is wrong with the code, will fix it this weekend and push new version. In meantime you could increase your soft limits?

alfredabounasr commented 7 years ago

Thank you very much and I appreciate your hard work.

On Fri, Apr 28, 2017 at 4:59 PM Gerrit Visser notifications@github.com wrote:

Unfortunately images don't come through when responding by email :-( Knowing it is error:15 now I know it is Jog command that it fails on.

I know what is wrong with the code, will fix it this weekend and push new version.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gerritv/Grbl-Panel/issues/108#issuecomment-298005179, or mute the thread https://github.com/notifications/unsubscribe-auth/Aamsda03D6x_2VVgApG31CmZgC6WTTA0ks5r0fCpgaJpZM4NFJVQ .

gerritv commented 7 years ago

I added 3 missing error messages. Fixed in 1.0.9.14