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

PySWMM v1.0 #262

Merged
merged 47 commits into from
Jan 12, 2021
Merged

PySWMM v1.0 #262

merged 47 commits into from
Jan 12, 2021

Conversation

jennwuu
Copy link
Contributor

@jennwuu jennwuu commented Jan 4, 2021

This pull request refactors pyswmm swmm5 module to use swmm-toolkit package to interact with model data from SWMM API. Moving forward, PySWMM is only compatible with Python 3+ 64bit.

Setup Github Action
Closes #263 The ability to use different SWMM .dll is also removed.
Closes #264 swmm5 module refactor
Closes #265 Repository cleanup

@jennwuu jennwuu self-assigned this Jan 4, 2021
@jennwuu jennwuu requested a review from katmratliff January 4, 2021 19:56
@jennwuu jennwuu added this to the v1.0.0 milestone Jan 4, 2021
@@ -17,7 +17,8 @@
from pyswmm.system import SystemStats
from pyswmm.raingages import RainGages, RainGage

VERSION_INFO = (0, 6, 3, 'dev0')
VERSION_INFO = (1, 0, 0, 'dev0')
Copy link
Member

Choose a reason for hiding this comment

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

@jennwuu can you confirm the versions here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bemcdonnell the SWMM version?

Copy link
Contributor Author

@jennwuu jennwuu Jan 5, 2021

Choose a reason for hiding this comment

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

@bemcdonnell @michaeltryby This version information should removed since swmm is from swmm-toolkit package

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bemcdonnell
Copy link
Member

Please make sure to clearly note the deprecation of the DLL picker feature in the release notes

Copy link
Contributor

@michaeltryby michaeltryby left a comment

Choose a reason for hiding this comment

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

I approve these changes.

Copy link
Collaborator

@abhiramm7 abhiramm7 left a comment

Choose a reason for hiding this comment

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

Looks great!

Copy link
Contributor

@katmratliff katmratliff left a comment

Choose a reason for hiding this comment

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

Nice work!

@jennwuu jennwuu merged commit ed6de57 into master Jan 12, 2021
@jennwuu jennwuu deleted the dev-swmm-toolkit branch January 12, 2021 15:09
@michaeltryby
Copy link
Contributor

🎈🥳🎉

@bemcdonnell
Copy link
Member

💃🕺💃🕺💃🕺💃🕺💃💃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Repository clean up Refactor PySWMM to use swmm-toolkit package Deprecate DLL selector function
7 participants