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

Remove array index offset #50

Closed
wants to merge 1 commit into from
Closed

Conversation

ecbrown
Copy link
Contributor

@ecbrown ecbrown commented May 6, 2023

Fix proposed in #41

@ecbrown ecbrown mentioned this pull request May 6, 2023
@vmbecerra
Copy link
Contributor

Dear Eric

Thank you so much for your suggestion.

I have realised that the *_propagate functions are difficult to use and fairly restrictive. Moreover, as you noted they are not working well. There is actually a better way of propagating a user-supplied control input by solving an optimal control problem with a special objective functional. This allows to include path constraints, which were not possible to consider using the 'propagate' functions. I will document this alternative approach instead and remove the propagate functions, so I will not approve this particular merge request as it will not be needed anymore.

Best wishes

Victor.

@vmbecerra vmbecerra closed this Sep 2, 2024
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