INABILITY TO READ IN OBERLIN NETWORK USING WNTR

[uthmankareem]

Summary
Describe your question.
I am trying to read in an inp file for oberlin network on python jupyter. I have used the same command to read in modena network and it worked. However, when I try to read in oberlin network, it gives attribute error: None type has no attribute 'factor'

Attached is the error screenshot.

Thank you.

Environment
[Optional] Provide information on your computing environment.

• Operating system: Windows OS 10
• Python version:
• WNTR version: 2.2

[kaklise]

Does the INP file include flow units (i.e. GPM)? EPANET might assume default units, but it looks like WNTR requires that field. One way to populate the INP file with default values is to load the file into the EPANET GUI and resave the file.

@dbhart it would be helpful to update io.py to catch this issue before flow_units is used elsewhere.

[uthmankareem]

Hello, Thank you for the help. I suspected the error had to do with flow by reading the error report but I did not know how to address it. Also, on doing a manual check on the file using txt, I did not see it having a reservoir label. To be sure I get you correctly, is it just enough to load the inp file into epanet software, save and rename it? Also, I do not understand what you mean by updating io.py. How do I do this and what is io.py? Attached is the inp file and the jupyter notebook used. The network zip folder came with many dat, mao and lst files separately but I am not sure if I need all those so I only used the only ino file in it. Thank you. Best regards, Uthman Kareem

…

On Mon, May 1, 2023 at 10:22 AM Katherine Klise ***@***.***> wrote: Does the INP file include flow units (i.e. GPM)? EPANET might assume default units, but it looks like WNTR requires that field. One way to populate the INP file with default values is to load the file into the EPANET GUI and resave the file. @dbhart <https://github.com/dbhart> it would be helpful to update io.py to catch this issue before flow_units is used elsewhere. — Reply to this email directly, view it on GitHub <#348 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AL6ZC4MBGWNDWJ2ZL3V63X3XD7IKVANCNFSM6AAAAAAXQJ6TH4> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[kaklise]

That is correct, loading the INP file into the EPANET GUI, then saving it under a new name should add the missing default entries. The comment about io.py was for one of our developers.

[uthmankareem]

Hello, Thank you. I did as advised yesterday and it worked. Best regards, Uthman Kareem

…

On Tue, May 2, 2023, 10:24 AM Katherine Klise ***@***.***> wrote: That is correct, loading the INP file into the EPANET GUI, then saving it under a new name should add the missing default entries. The comment about io.py was for one of our developers. — Reply to this email directly, view it on GitHub <#348 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AL6ZC4MIEC3YNZVVO4PBFF3XEERKJANCNFSM6AAAAAAXQJ6TH4> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[dbhart]

Does the INP file include flow units (i.e. GPM)? EPANET might assume default units, but it looks like WNTR requires that field. One way to populate the INP file with default values is to load the file into the EPANET GUI and resave the file.

@dbhart it would be helpful to update io.py to catch this issue before flow_units is used elsewhere.

Yes, it would, I will try to catch that.

[uthmankareem]

Hello, I just opened an issue on key error 0. I guess that is as a result of partial results being returned because the simulation did not converge. However, I am not sure this results are accessible by any means. I will be glad if you can help me look at the issue I opened. That is the code below. "pressure_above_threshold = wntr.metrics.query(pres, np.greater,threshold) pressure_above_thresholdT = pressure_above_threshold.T pressure_below_threshold_lst = pressure_above_thresholdT.index[pressure_above_thresholdT[0] == False].tolist()" I also have a pump overflow issue. I am not sure if that is something I need to worry about. Best regards, Uthman Kareem

…

On Fri, Jun 9, 2023 at 4:13 PM Katherine Klise ***@***.***> wrote: Closed #348 <#348> as completed. — Reply to this email directly, view it on GitHub <#348 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AL6ZC4OSA6K3UZVZ6UK46ELXKOGWXANCNFSM6AAAAAAXQJ6TH4> . You are receiving this because you authored the thread.Message ID: ***@***.***>