Skip to content
This repository has been archived by the owner on Apr 6, 2023. It is now read-only.

Clean up of Surrogates examples after move to apps #108

Merged
merged 3 commits into from
Mar 21, 2022

Conversation

andrewlee94
Copy link
Member

Corresponds to IDAES/idaes-pse#759

Note that the RIPE examples are currently non-functions and are skipped. This really needs to be fixed by someone, as otherwise these really should be removed from the repo.

Proposed changes:

  • Delete outdated ALAMO examples to avoid confusion with new interface. The old example was trivial anyway.
  • Update imports in RIPE and HELMET examples.

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@andrewlee94 andrewlee94 self-assigned this Mar 21, 2022
@andrewlee94 andrewlee94 added bug Something isn't working Priority:Normal Normal Priority Issue or PR labels Mar 21, 2022
@andrewlee94 andrewlee94 added this to In progress in IDAES 2.0 via automation Mar 21, 2022
@andrewlee94 andrewlee94 moved this from In progress to Review in progress in IDAES 2.0 Mar 21, 2022
@lbianchi-lbl lbianchi-lbl merged commit 2a61627 into IDAES:main Mar 21, 2022
IDAES 2.0 automation moved this from Review in progress to Done Mar 21, 2022
@andrewlee94 andrewlee94 deleted the surrogate_cleanup branch March 21, 2022 20:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working Priority:Normal Normal Priority Issue or PR
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants