-
Notifications
You must be signed in to change notification settings - Fork 38
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
Unlock pyomo version(s) #142
Conversation
This PR was ready to go until the ravenframework change. Need to wait for #140 before attempting to merge. |
Job CentOS 8 on 98c71cd : invalidated by @PaulTalbot-INL ravenframework fix, rerunning tests |
@dgarrett622 This passed tests but is marked with "do not merge"; any opposition to removing that flag? |
@PaulTalbot-INL no problems with removing the flag and merging. I tried removing the flag earlier, maybe it didn't stick. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good and should not affect functionality or results.
Pull Request Description
What issue does this change request address?
Continues #74
What are the significant changes in functionality due to this change request?
Continuing discussion from #124 allows upgrading pyomo from 5.7 as a dependency
For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.