Skip to content
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

Implement "actuator activation" to compensate drive latency #11

Merged
merged 5 commits into from
Jan 31, 2023

Conversation

DaniEll-AT
Copy link
Contributor

@DaniEll-AT DaniEll-AT commented Jan 4, 2023

Implement "actuator activation" like discussed in #7

One implementation detail i am not sure about is, wether "actuator activation duration" should be considered for open_net_duration_/close_net_duration_. I am leaning towards adding that (not done currently). I implemented it this way.

@DaniEll-AT DaniEll-AT mentioned this pull request Jan 4, 2023
@DaniEll-AT DaniEll-AT marked this pull request as ready for review January 4, 2023 16:15
@DaniEll-AT
Copy link
Contributor Author

@pawelma and @bruxy70 I think the code change in this Pull Request is consistent with the way the test-branch works. I use it quite some time now, it makes a major difference in tilt-precision. Please review/merge it.

@bruxy70 bruxy70 merged commit b1b8ba5 into bruxy70:test Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants