Correction of Error Message in EcoRoof when User Does Not Use CTF Solution Algorithm #11095
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In order to run an EcoRoof simulation, the user must run a CTF based solution for the surface(s) that is/are green roof(s) or EcoRoof(s). Currently, E+ provides a severe error when the user requests an EcoRoof simulation but does not use the CTF solution algorithm. However, after an error message is produced, the simulation continues on. The improvement here is that now E+ will fatal out. In addition, more details are provided to the user as part of the error message that reminds the user of what syntax changes the solution procedure including for surfaces and also point to the surface that has a problem. Unit tests also verify that the solution must be CTF or an error will be produced.
Pull request overview
Description of the purpose of this PR
This PR provides the fix and unit test to solve the defect with E+ where the solution continues when an EcoRoof is defined and the surface does NOT use the CTF solution algorithm.
Pull Request Author
Reviewer