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

betts 10 113, new #291

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Peter230655
Copy link
Contributor

@Peter230655 Peter230655 commented Dec 30, 2024

I pushed this from a wrong branch earlier.

This is a very simple example how to handle inequalities of the form: function(state_variables, control_variables, parameters) >= 0
I simply define a new state variable, say, J, and set J = function(state_variables, control_variables, parameters) as additional equation in the equations of motion.
Then, in the bounds I set: { J: (-infinity , 0.0) }

I made similar PRs before, but none this simple.
I guess, a matter of taste, which, if any at all, should make it into examples-gallery.

@moorepants moorepants closed this Jan 27, 2025
@moorepants moorepants reopened this Jan 27, 2025
@moorepants
Copy link
Member

You successfully merged the upstream master and pushed this. And the build is complete. Nice job!

@Peter230655
Copy link
Contributor Author

Peter230655 commented Jan 27, 2025

You successfully merged the upstream master and pushed this. And the build is complete. Nice job!

I had a LOT of help!
Question: Do you know a person dumber than me in git / github?
Answer; NO!! :-)

  1. In one or two PRs I proposed changes to the class Problem. Would this work the same way, or would I have to manually correct?
  2. Should I push all PRs for examples-gallery this way, now that I know how to do it? ( I will do a few more in any case, just to make sure I know wehat to do)

@Peter230655
Copy link
Contributor Author

All my PRs should be up-to-dater now. :-)

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