I made a subtle mistake in the code that chooses whether to use trapezoidal or ramped acceleration, which can lead to strange behavior when using spindle-synced motion. The result was that short segments would sometimes be ramped up over a very long time (~20 seconds). This fix corrects the final velocity estimation so that this doesn’t happen.
评论 (2)
#2 – robEllenberg 于 2015-02-09
Sorry! That PR was meant to be for the machinekit repo (so I closed it
without merging). I pushed a branch to the official repo with this fix
(based on a recent 2.7).
-Rob
On Mon, Feb 9, 2015 at 5:49 PM, Sebastian Kuzminsky < notifications@github.com> wrote:
> PRs for linuxcnc are welcome, just please make sure they’re based on some
> reasonable commit in our repo.
>
> —
> Reply to this email directly or view it on GitHub
> https://github.com/jepler/linuxcnc-mirror/pull/8#issuecomment-73608001.
![[gnea/grbl-Mega Issue#1] $ command extentions](https://www.grblhal.com/wp-content/themes/gitphp/timthumb.php?src=https://www.grblhal.com/wp-content/themes/gitphp/assets/img/pic/2.jpg&h=110&w=185&q=90&zc=1&ct=1)
![[gnea/grbl-Mega Issue#2] Better interrupt priorities](https://www.grblhal.com/wp-content/themes/gitphp/timthumb.php?src=https://www.grblhal.com/wp-content/themes/gitphp/assets/img/pic/3.jpg&h=110&w=185&q=90&zc=1&ct=1)
#1 – SebKuzminsky 于 2015-02-09
PRs for linuxcnc are welcome, just please make sure they’re based on some reasonable commit in our repo.