-
Notifications
You must be signed in to change notification settings - Fork 12
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
DFlee feature porting #81
Comments
The flood_level data structure is now in place, and properly being read in and stored on the locations. @alirezajahani60 I do not have dynamic flood level properties for link attributes yet. Is this something you want to have present in the first version of DFlee, or would you prefer to have an early test version with only location flood levels for now? Adding dynamic flood level properties for links could take a few days of development effort, i.e. ~2 weeks delay in actually code delivery time. |
Btw. take special note of https://github.com/djgroen/flee/blob/master/tests/dflee.py This is what I initially wanted you to write, but I put it together myself now. It will be handy for you, as you can see how the DFlee functionality is called and used. The idea is of course that DFlee will not require a separate runscript. Simple changes in simsetting.yml should suffice for it. |
@djgroen Thanks Derek. I think for the beginning we can have at location level and add dynamic flood level for links later. I still need help to understand the relationship between the runscript and test_dflee.py! If I understand correctly, by changing the simsetting.yml, I can simply execute the runscript and the test_dflee.py, which is in flee directory would read the input files and the flee.py does the rest of simulation? |
@alirezajahani60 Partially correct! Let me elaborate :) "If I understand correctly, by changing the simsetting.yml, I can simply execute the runscript"
"and the test_dflee.py, which is in flee directory would read the input files and the flee.py does the rest of simulation?""
|
Documentation is written but not pushed to the repo yet. I will leave this ticket open during the testing phase. |
flood_level.csv
.And then:
The text was updated successfully, but these errors were encountered: