-
-
Notifications
You must be signed in to change notification settings - Fork 260
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
Update TEST_SPEED.cfg #80
base: main
Are you sure you want to change the base?
Conversation
Vykori
commented
Jun 11, 2023
•
edited
Loading
edited
- Added extra safety measures
- Fixed typos and indentation inconsistencies
- Added extra movements to test moving one way, and then immediately back again
- Tested on Voron 0.2
- Added extra safety measures - Fixed typos
Thanks for the contribution. Some of the changes were intentional behaviors:
The safe speed and the additional stallguard delay look good. I might go even lower (max_velocity / 3 or 4 maybe?) but that's debatable |
Ooh good insight. I'll put those g-codes back in and re-submit |
- Fix regressions pointed out by Ellis AndrewEllis93#80 (comment)
- Added additional movements to test toolhead moving one way then back again in all combinations
I've added extra moves to the box and diagonal patterns to additionally test moving one way, then immediately moving back again. My thinking is the |
Fixed an accidentally misplaced G0 movement in both the large and small box pattern
Should this be two separate PRs? I'm new to Git things... |