-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[RPP] Make new deceleration on cancelation an option #4238
Comments
Maybe we could just use |
Hi @SteveMacenski, I am new to open source contributions and would like to work on this, wanted to check in if someone is already working on this? Thanks |
Let's say that a developer sets the value of the proposed parameter, call it |
I would say 0.0, as before the (also consider not using a new parameter but just |
@tanmay-dhasade no one is working on it yet, this would be a great first contribution! |
Merged! |
Create a parameter for if we should cancel using deceleration (default true), then use that in the logic introduced in #4136 to bypass if needed
Finally, add the new parameter to documentation & add the new param to the RPP deceleration migration guide entry
The text was updated successfully, but these errors were encountered: