search

USEPA / SWMM-EPANET_User_Interface

User interface for the Stormwater-Management-Model
112 stars 67 forks source link

Fix SWMM summary tables #236

Closed barrc closed 6 years ago

barrc commented 6 years ago

Fixes for SWMM summary report tables. Closes #141.

michaeltryby commented 6 years ago

@barrc Its great that you are digging in on this! Since the team is most likely going to be you, me, and @woohn, we should be reviewing each others work. In the future assign us as reviewers. Sound good?

barrc commented 6 years ago

@michaeltryby sounds good!

michaeltryby commented 6 years ago

We should also think about adding unit tests as we go, to insure that things we fix are actually fixed / stay fixed. Make sense?

michaeltryby commented 6 years ago

@barrc Hold off on merging for the moment. Ok. I don't like to work directly in master.