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

pd documentation #95

Merged
merged 12 commits into from Jul 17, 2023

Conversation

tohsin
Copy link
Contributor

@tohsin tohsin commented Jun 20, 2023

Improved Documentation for PD controller in the tutorial

Further break down of what a PD controller was, was given, as well as explanations of the choice of equation we choose to implement.

Copy link
Collaborator

@balisujohn balisujohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines which say "references." maybe included by accident. Otherwise, ready for merge :^)

# is approaching the target position. When the agent is moving quickly towards the target, the
# derivative term will be larger, contributing to a stronger corrective action from the controller.
# On the other hand, if the agent is already close to the target and moving slowly, the derivative term will be smaller,
# resulting in a less aggressive control action.
# references.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These lines are probably a typo.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I suspect I was trying to justify why we had a pure velocity term as opposed to a first order differential error rate.
Yeah i think the reference can go

@balisujohn
Copy link
Collaborator

LGTM :^)

@balisujohn balisujohn merged commit 275ded1 into Farama-Foundation:main Jul 17, 2023
12 checks passed
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.

None yet

2 participants