-
Notifications
You must be signed in to change notification settings - Fork 54
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
Reformat qlat inputs #595
Reformat qlat inputs #595
Conversation
raise(RuntimeError("Output binary qlat folder supplied in config does not exist")) | ||
elif len(os.listdir(binary_folder)) != 0: | ||
raise(RuntimeError("Output binary qlat folder supplied in config is not empty")) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All files in binary_folder should be cleanly deleted for a new simulation. When Delete option is used, one hidden file, i.e., .ipynb_checkpoints, still remains to give the error of line 53. It is recommended to use "rm -r * .*" to cleanly delete all files. Other than this, this PR successfully ran with nextgen-generated csv files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This new change at line 52 solved my initial issue. Well done!!
raise(RuntimeError("Output binary qlat folder supplied in config does not exist")) | ||
elif len(os.listdir(binary_folder)) != 0: | ||
raise(RuntimeError("Output binary qlat folder supplied in config is not empty")) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This new change at line 52 solved my initial issue. Well done!!
Added capability to convert nexus qlat csv files output by ngen (single file per nexus with all timesteps) into hourly parquet files (one file for every hour containing all nexus points) that conforms to t-route's current method of reading qlat input data.
Additions
Removals
Changes
Testing
Screenshots
Notes
Todos
Checklist
Testing checklist
Target Environment support
Accessibility
Other