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

Integrate desalination models #311

Merged
merged 130 commits into from
Jun 4, 2024

Conversation

Sakshi21299
Copy link
Contributor

@Sakshi21299 Sakshi21299 commented Mar 18, 2024

Summary:

This PR adds integrated optimization formulation for the MEE-MVR process along with jupyter notebooks with 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.md 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.

Sakshi21299 and others added 30 commits January 1, 2024 15:56
Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
…_notebook.ipynb

Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
…_notebook.ipynb

Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
…_notebook.ipynb

Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
…_notebook.ipynb

Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
…_notebook.ipynb

Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
…_notebook.ipynb

Co-authored-by: Travis Arnold <travis.arnold17@gmail.com>
@Sakshi21299
Copy link
Contributor Author

@tarnold17 The files for the MD unit are necessary for the jupyter notebook with integrated MD at desalination sites. When 307 gets merged I think they don't need to be here

Copy link
Contributor

@tarnold17 tarnold17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a few questions and comments. I've also just pushed a few commits cleaning some things up. Namely, I have removed the files MD_PARETO.ipynb and MD_single_stage_continuous_recirculation.py from the repo, as these will be contributed in #307. This does mean that #307 now needs to be merged before this PR can merge.

pareto/case_studies/integrated_desalination_demo.xlsx Outdated Show resolved Hide resolved
pareto/tests/test_cm_qcp.py Outdated Show resolved Hide resolved
pareto/models_extra/Integrate_desal/models/qcp_desal.py Outdated Show resolved Hide resolved
pareto/tests/test_integrated_optimization_mvr.py Outdated Show resolved Hide resolved
@Sakshi21299
Copy link
Contributor Author

I have a few questions and comments. I've also just pushed a few commits cleaning some things up. Namely, I have removed the files MD_PARETO.ipynb and MD_single_stage_continuous_recirculation.py from the repo, as these will be contributed in #307. This does mean that #307 now needs to be merged before this PR can merge.

Yes that means #307 needs to merge before this

@tarnold17
Copy link
Contributor

I have a few questions and comments. I've also just pushed a few commits cleaning some things up. Namely, I have removed the files MD_PARETO.ipynb and MD_single_stage_continuous_recirculation.py from the repo, as these will be contributed in #307. This does mean that #307 now needs to be merged before this PR can merge.

Yes that means #307 needs to merge before this

#307 was merged earlier this week, so we are good there!

Copy link
Contributor

@JavalVyas2000 JavalVyas2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@tarnold17 tarnold17 merged commit 37e64e5 into project-pareto:main Jun 4, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:High High Priority Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants