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

Allow multiple outfalls to be listed in a single timeseries file #31

Open
MitchHeineman opened this issue Jan 28, 2020 · 6 comments
Open
Assignees
Milestone

Comments

@MitchHeineman
Copy link

MitchHeineman commented Jan 28, 2020

I'm working on a model that has over 100 outfalls. Rather than requiring a unique file for each outfall, it would be more elegant to expand the timeseries file format to allow an optional ID (e.g. following the value field) and thus allow the user to use a single boundary conditions file. No GUI change would be needed.

Alternatively, the file format could follow the structure of calibration files in the EPA GUI, where the location ID is listed on a header line and datetime/value pairs follow.

EPA SWMM has at least five timeseries formats: internal timeseries (ID m/d/y h:m value), timeseries file (m/d/y h:m value), GUI calibration file timeseries (same as external timeseries but with ID header), routing interface file (header followed by ID y m d h m s value), and rain gage (like routing interface but without header or seconds). Despite all these options, it does not accommodate y/m/d and d/m/y formats and does not allow a multi-location outfall file as I've suggested here.

@dickinsonre
Copy link

A few general tools to convert the DAT file format back on forth would also be good. I especially like the format of the calibration file - an add on to convert this to other formats might be useful.

@cbuahin
Copy link
Collaborator

cbuahin commented Aug 25, 2023

What about explicit support for csv files? They could named as one would name a timeseries file and could have multiple columns. Each column can be assigned to an object using the convention TIMSERIES_NAME:COLUMN_NAME.

@dickinsonre
Copy link

Good - that can also work for inflow time series

@MitchHeineman
Copy link
Author

@cbuahin Do you expect to support the proposed .csv format in version 5.3? Is that the next planned version, or are you expecting 5.2.5 will be next?

@dickinsonre
Copy link

I think Caleb said 5.3 would be the next version number at the ICWMM 2024 conference last week.

@cbuahin
Copy link
Collaborator

cbuahin commented Mar 6, 2024

@MitchHeineman, after considering things internally, we felt enough new features were being considered to warrant bumping the minor version number.

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

No branches or pull requests

4 participants