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

Source Code Files Not Up To Date With EPA Files #7

Open
LRossman opened this issue Apr 8, 2021 · 3 comments
Open

Source Code Files Not Up To Date With EPA Files #7

LRossman opened this issue Apr 8, 2021 · 3 comments
Assignees

Comments

@LRossman
Copy link

LRossman commented Apr 8, 2021

The source code files in this repository are out of date when compared with those published by US EPA on their web site . The latter files are from 2010/2011 while those in this repository are from 2007/2008. The newer EPA files include a fix for the "parsing exponents" problem noted in issue #1 . When comparing the two sets of files I found that 19 of 30 files had been updated in the EPA release.

@samhatchett
Copy link
Member

indeed! thanks for catching this. I will open a new branch for that version and see how extensive the diff is. Would you be willing to review a PR and make sure we are not breaking anything with a merge?

@samhatchett
Copy link
Member

I skimmed over the changes and agree with your assessment, @LRossman - the version bump looks like it affects the exponent bug and the JIT compilation for the most part. All other changes look to be formatting/whitespace and some error checking. The biggest issue I have with attempting a merge is the lack of a unified build configuration. CMake is great for this - so we might think about swapping in a single cmake config as a part of this update?

@samhatchett samhatchett self-assigned this Apr 8, 2021
@LRossman
Copy link
Author

LRossman commented Apr 8, 2021

@samhatchett I have cmake scripts ready to go (see attached files). They assume the project has sub-directories src, run, and include. Re-name the attached CMakeLists_1.txt to CMakeLists.txt and place it in the top-level directory. Rename CMakeLists_2.txt to CMakeLists.txt and place it in the run directory, along with msxmain.c. The include directory should contain epanetmsx.h and epanetmsx.def. I think it should also include epanet2.h, epanet2_enums.h, and epanet2.lib from the OWA-EPANET v2.2 repository.
CMakeLists_1.txt
CMakeLists_2.txt

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

No branches or pull requests

2 participants