-
-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] Mesh inset settings do nothing #143
Comments
how about the Physical Settings, under the Control menu? shouldn't this be adjusted instead of the Mesh Inset? |
@classicrocker883 I set the X max on physical settings to 180, stored settings, and it still crashes into the lcd screen. |
so I'm a bit confused. physical settings are one thing, and mesh inset are another. being able to set the bed dimensions, X/Y Bed Size, and X/Y Min/Max should be working normally. if you set it to max 180, what then is the actual limit that it ends up crashing? |
@classicrocker883 The mesh X maximum was set to 199.0 in the Mesh Inset settings so I guess that's the limit. It never crashed into the LCD screen when I set the mesh X maximum to 199.0; I was able to create the mesh in the past. |
I had made some changes regarding mesh_inset code, how it is defined. I was able to test with probe firmware fine, so I have a suspicion of what it can be. could you tell me which firmware release did work fine last that you know? I will Update: here you can unzip this and flash this update see if this works. please let me know if this fixes the issue. (dont worry about the file name just place into the SD card like normal) |
@classicrocker883 I think it was the November release of last year, I couldn't remember it's been a while since I updated the firmware. I'm using a 3x3 mesh and I tested Aquila_GD32_MM-ProUI-EX-test.zip. The 1x3 point of the mesh (the last X point until it begins the next Y set) is at the middle of the bed. The 2x3 is maybe 1 inch or 1/2 inch away from the 1x1 which the 1x1 point is right where the X home is. Adjusting the X maximum mesh inset to 220 (when it was 210) does not make a difference, the 1x3 point is still at the middle of the bed. |
okay, give this one a try, that should do it. |
@classicrocker883 Same. The 3rd probe point is at the middle of the bed. |
Aquila_GD32_MM-ProUI-EX-test3.zip heres a couple more tests. I went back and took a look through the code from when you say it worked, I couldn't find anything that stood out, but made a couple changes I thought would work. if you can, make a quick video and upload it, that'll help me understand whats actually happening. |
@classicrocker883 Neither of the two made a difference. |
I don't know if this helps, In the physical settings menu, I set the max bed size to 230 for both x and y and the bed size to 220 for both of them. I then set the mesh inset to 10.0 for each. I'm using a bltouch, not manual, but maybe these settings will have the head move to the right positions for the mm. |
I'm not sure what's going on, I will have to perform some tests to figure it out. because I had updated the mesh inset code, I tried reverting to what it was originally, so i'll have to do some git bisect I think all the way back to November, so thats gonna take some time, trial and error. looking back and comparing the code didn't give much clue. it is just odd how the mesh is appearing to be almost exactly half of what it should be. for now I suppose either use the last build release which worked, or work with what you got. because technically with a good and level bed you should not need any active bed leveling, that is even if there is slight variation +/-0.10, that isn't really significant enough to effect the first layer. I know with mine I can turn active leveling off and there's no noticeable difference. just hang in there i'll try and figure this out. |
@classicrocker883 It could've been either the November or October release, they may have the same logic so I doubt it makes a difference. |
Im having a similar issue when using the June UBL PRO UI. my Aquila has an ender extender kit so my bed is 400mm x 400mm. Ive set my physical settings and mesh settings to 400mm. When I home my Z axis, it correctly goes to the center of the bed but when I try to make a mesh, it probes based on the default ~220X220 bed size instead. |
@RCP1990 have you used an October or November version? if so, did it work normally then? that is interesting since it is UBL, yet works fine on default bed sizes. |
I have not. ill install the November and test it out. |
what .bin file do you use? chip and version and such? because I may have found something and I can provide and updated firmware for you to test. |
GD32-MM-ProUI-EX-July-Test1.zip |
@classicrocker883 UBL works as expected on the November release. (im using a G32 Aquila and the UBL Pro UI 7x7 bin, for the record) |
@classicrocker883 Same problem with 3x3 mesh as #143 |
@classicrocker883 No change with JULY 2024 2.1.3f-7. |
well I hate to say it, but it would be great it you could try like the December, January, Feb... and so on, from the working November (or is it November Rev. 1?) until you get to the one that doesn't work. that would be super helpful. even after comparing then from now I still cannot find why it would the mesh would be like in half for example, I have suspicion in February. Try this:
then that will at least narrow it down. again, above is where I would start. please let me know if you test these out. Here is a list of the other releases if that helps out:
|
I really appreciate the help trying to solve this. I made just one more test file which is really optional, it would be much better going month by month releases and letting me know the point it goes from working to not working. GD32-MM-ProUI-EX-July-Test2.zip This is a separate test which I took November and copied the current dwin.cpp (main UI display file). this should work - but if this doesn't work then it will mean its something in this file and should narrow it down. actually it's not exactly the November release, but close enough. again at this point there has been lots of changes since November so finding where one release works and the next doesn't is the best course of action and should really help solve the issue. |
@classicrocker883 The GD32-MM-ProUI-EX-July-Test2.zip had no change. The February Rev. 1 build probes the entire bed properly using the 3x3 like it had in the past. |
so youre saying Test2 works as it should, or no? here is another test which I added the workaround for the mesh inset, which is present in November, and that I had previously removed thinking I solved the issue before. if this works then thats gotta be it. |
@classicrocker883 Sorry for the wait, GD32-MM-ProUI-EX-July-Test3.zip works as it should. |
@classicrocker883 I'll test when I come back Wednesday. |
@classicrocker883 |
Test3 working is good news. I will be posting a new release soon, working on a couple things - extra firmware compilations. |
@Arson31 if you can let me know if that is also working as it should |
@classicrocker883 the problem's still there using Aquila_GD32_MM-ProUI-EX-09-19.bin. |
ok, so you sure it was Test3 from earlier that worked? because that is what I went with for changes |
@classicrocker883 Ok, something strange is happening. When I tested the two firmwares 2 weeks ago:
That was true before, but now I can't get either of them to probe correctly. Here's what I did:
I flashed back to GD32-MM-ProUI-EX-July-Test3.zip here's what I did with that:
If the order I handle these doesn't matter then I don't know how to go about this. Does reformatting the SD card before every flash make a difference? I've never done so with any of these tests but going back to GD32-MM-ProUI-EX-July-Test3.zip without reformatting the SD card fixed the probing. Also the latest release https://github.com/classicrocker883/MRiscoCProUI/releases/tag/2.1.3f-9 had violent movements when auto homing. Never had that before. |
here is a new test file. I moved the values to a struct to follow the current head fork of MRiscoCProUI. before it was a simple workaround, this change is better so hopefully it works.
it seems like your steps are correct, but it shouldn't matter changing any settings specifically after every flash. you can if you want for consistency's sake, but having to remove the card + restarting wouldn't make a difference.
The SD card itself doesn't need to be formatted every time. just once ever. |
@classicrocker883 I tried GD32-MM-ProUI-EX-Sept-Test1.zip and it did not probe properly. I noticed after storing settings and a restart the steppers react violently, The carriage slams the X switch when auto-homing. |
that is strange. but the previous August Test3 worked the best? no other problems? I might revert back to that, because Marlin should have fixed the issue with the motors shaking. 😔 |
I've uploaded a new release for October, the issue should be fixed, @Arson31 hopefully it works let me know |
@classicrocker883 I've tried it and had the same problem as the previous builds. The violent stepper movement went away though. I'm sticking to GD32-MM-ProUI-EX-July-Test3.zip. |
I think what was happening was the order which the mesh inset ( this would explain why it would only be at the middle of the bed since but what baffles me is I implemented both November and February ways in the tests, and I dont get why it worked then, but not any other time. so, heres one more test you can try which I explicitly include the code that edits the mesh inset. |
@classicrocker883 Same problem for GD32-MM-ProUI-EX-October-Test1.zip. |
im sorry this doesn't make any sense. how can something work before, and not this time? |
GD32-MM-ProUI-EX-October-Test2.zip well I moved where the mesh_inset is defined to a struct like how the recent original MRiscoC is, couldn't find anything else that stood out which would make it not be the correct mesh inset. I know it did work before for UBL so its odd to say why it wouldn't for Manual Mesh, and when it did work at one point. hopefully moving it this way makes a difference otherwise it could be something else happening with Manual Mesh and not with the other bed leveling types. |
@classicrocker883 GD32-MM-ProUI-EX-October-Test2.zip didn't work either.
I did not remove the SD Card. None of this may not matter, I'm just listing the steps of what I did exactly. |
GD32-MM-ProUI-EX-October-Test3.zip |
@classicrocker883 That did it! It probes correctly now. Thanks! |
Did you test the latest release build?
Yes, and the problem still exists.
Bug Description
Updated to MAY 2024 Rev.2 build. (Aquila_GD32_MM-ProUI-EX-05-23.bin)
I set the X maximum to 199.0 because I'm using a Satsana shroud for the Haldis Red Lizard Q1 like I have with the previous builds. After setting and saving the settings the shroud still crashes onto the LCD screen mounted vertically to the right of my Aquila.
I set the X maximum to 160 (which should be dead center) and the shroud still crashes into the screen.
Printer Model
Voxelab Aquila
Model Type
OG
Your Mainboard
Aquila GD32
Other Mainboard Type
No response
Add-ons that could be involved
Satsana shroud for the Haldis Red Lizard Q1
ProUI-EX | Extra Features?
Yes
Bed Leveling
MM - Manual Mesh Bed Leveling
Did you include your own configuration files?
Configuration.h
andConfiguration_adv.h
.Additional information & file uploads
No response
The text was updated successfully, but these errors were encountered: