Skip to content
This repository has been archived by the owner on Aug 27, 2023. It is now read-only.

Acceleration config for each axis separately #294

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wsowa
Copy link
Contributor

@wsowa wsowa commented Feb 12, 2018

In most printers at least Z and E axis have different mechanics than X and Y. Often X and Y differs as well. Therefore it makes sense to control acceleration of every axis separately. Indeed many other firmwares allow for such config. This change adds ability to configure acceleration for X,Y,Z and E independently

@Wurstnase
Copy link
Collaborator

Thanks for your contribution.

Your approach will work excellent on single axis movements. But unfortunately not on multiple axis movements.

Movements are dependent on each other. The one axis with the most steps per second will win. On this axis the speed will calculated. So when this axis has high acceleration and a tiny slower axis has much lower acceleration values, the tiny slower axis will accelerate with the high acceleration from the tiny faster axis.

So you need to check the acceleration in dda_clock and calculate a new acceleration for the move and the fast axis. This will not be constant anymore.

@Traumflug
Copy link
Owner

True, a valuable contribution. The Configtool part looks good.

To get the remaining part working for multiaxis movements, one has to calculate acceleration duration for each participating axis and take the longest one. Then calculate acceleration on the fast axis using this longest duration.

@wsowa
Copy link
Contributor Author

wsowa commented Feb 14, 2018

I had a feeling it was too easy. I’ll get back to you with more changes. Thanks for the hints where to start.

@triffid
Copy link
Collaborator

triffid commented Feb 14, 2018 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants