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

Typo in System.py Module #162

Closed
Jiadalee opened this issue Jul 2, 2018 · 1 comment · Fixed by #163
Closed

Typo in System.py Module #162

Jiadalee opened this issue Jul 2, 2018 · 1 comment · Fixed by #163

Comments

@Jiadalee
Copy link
Contributor

Jiadalee commented Jul 2, 2018

Two errors in System.py module:

In line 22, pyswmm package doesn't include the SystemFlowRouting. In this way, the syntax of "from pyswmm import Simulation, SystemFlowRouting" is suggested to be "from pyswmm import Simulation, SystemStats" if we want to extract system statistics.

In line 27,‘’Simulation' has been demonstrated as ‘’sim”. Thus, the syntax of "for step in simulation" should be "for step in sim".

@bemcdonnell
Copy link
Member

@Jiadalee, thanks for spotting the issues in the docs!

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

Successfully merging a pull request may close this issue.

2 participants