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

Set the branch feature-2dflood up to date with the main branch #407

Open
wants to merge 875 commits into
base: feature-2dflood
Choose a base branch
from

Conversation

lrntct
Copy link

@lrntct lrntct commented Mar 16, 2025

This PR includes all the commit made so far on the main branch, plus some to fix the tests.
Related to #111

cbuahin and others added 26 commits June 22, 2023 12:25
Use of wrong variable for iteration: Addresses pyswmm#128
Adding Hot Start File save-during-simulation feature
Default values for two analysis options were changed to make them compatible with what the EPA SWMM GUI uses.
@bemcdonnell
Copy link
Member

@lrntct do you want me to approve as is? Looks like we’ve had tons of updates since this.

@lrntct
Copy link
Author

lrntct commented Mar 16, 2025

@bemcdonnell it's a ton of commits, yes. Most of them are already in main. I don't know if there is a plan to fix the CI. So far, I haven't run the NR tests. I have to figure out first how to run them.

@bemcdonnell
Copy link
Member

@lrntct would it be easier to rebase your code off main? One issue we are running up against with respect to the benchmarks is that GitHub actions deprecated our required version of visual studio. Therefore, technically we have been waiting for EPA to bump the release and update the benchmarks so we can match those.

@lrntct
Copy link
Author

lrntct commented Mar 16, 2025

I tried rebasing, but the issue is that I would need to fix the conflicts for every new commit on the dev branch. Since all the code moved location, this is very tedious. On top of that, I understand that rebasing is not advisable when the commit has been already pushed.
I am not a git guru, though, so I might be mistaken.

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.