-
Notifications
You must be signed in to change notification settings - Fork 76
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
NRTesting failing #117
Comments
@michaeltryby assuming someone has a clean install os Windows/Linux with nothing on it, what would be the steps to run the test suite. Any chance we could have those steps clearly described so we can post this inside the README of SWMM? |
The readme is located here. It needs to be updated. I will update it and then close this issue. |
@michaeltryby My bad, I read the readme too fast.
|
Well the thing is that nrtest only runs against windows executables? so on linux or OSX we can't run nrtest in the current setup? So we could maybe save the binary outputs ran on windows in another repository and use those as input for |
Platform agnostic. Michael currently has this implemented in Travis on SWMM and EPANET. |
Ok, then we really need to update those instructions |
@michaeltryby, I’m really excited about your nrtest contributions to this project! :-) |
@michaeltryby lets close after merge :-) |
This is still an issue. See #121 |
Nrtest is documented, there are SWMM instructions, there is a working example of it running under Travis CI. |
I followed the indications from the test README and ran into the following error:
Other combinations of json/inp give the same error.
OS: Ubuntu 17.10
swmm version 56fd8b2
Any idea?
The text was updated successfully, but these errors were encountered: