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

Add EPANETMSX support to WNTR #462

Merged
merged 126 commits into from
Jan 10, 2025
Merged

Add EPANETMSX support to WNTR #462

merged 126 commits into from
Jan 10, 2025

Conversation

dbhart
Copy link
Collaborator

@dbhart dbhart commented Dec 16, 2024

This includes the changes needed to support EPANETMSX with WNTR.

Summary

This feature adds EPANETMSX library support to WNTR. It also includes updates to the build process to allow MacOS-ARM support.

Acknowledgement

By contributing to this software project, I acknowledge that I have reviewed the software quality assurance guidelines and that my contributions are submitted under the Revised BSD License.

dbhart and others added 30 commits August 17, 2023 14:37
The class will be similar to the WaterNetworkModel in terms of its
structure and options.
Merge main into development branch
Update to add basic citation class
* Update to exception documentation

* Update documentation for EN errors
@dbhart dbhart requested review from kaklise and kbonney December 16, 2024 19:49
@dbhart
Copy link
Collaborator Author

dbhart commented Dec 16, 2024

I managed to mess up the PR by rebasing the last commits (it vanished) so this is the recreated. I do need to move shell/apple/libomp.rb still

@coveralls
Copy link

coveralls commented Dec 16, 2024

Coverage Status

coverage: 81.81% (-2.5%) from 84.341%
when pulling 83b1d6f on dbhart:msx
into 0badb0b on USEPA:main.

@dbhart
Copy link
Collaborator Author

dbhart commented Dec 17, 2024

Closing this PR and re-opening the original PR to keep the comments record.

@dbhart dbhart closed this Dec 17, 2024
@dbhart
Copy link
Collaborator Author

dbhart commented Dec 17, 2024

Well, PR #426 did not allow itself to be reopened anyway, so please look at that PR also for the history of comments on changes as they were prepped. Repoening.

@dbhart dbhart reopened this Dec 17, 2024
dbhart added 2 commits January 9, 2025 20:02
Also adds the ruby brew formula to the manifest for source distributions.
@dbhart
Copy link
Collaborator Author

dbhart commented Jan 10, 2025

@kaklise The last two pushes have fixed the outstanding issues with the msx_as files, moving the brew formula, and adding information on MacOS builds using homebrew to the installation section.

@kaklise kaklise merged commit c4e09e2 into USEPA:main Jan 10, 2025
53 checks passed
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.

4 participants