-
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
FOQUS not getting results for Aspen+ simulations ending with warnings. #268
Comments
fsoepyan
changed the title
FOQUS not getting results for A+ simulations ending with warnings.
FOQUS not getting results for Aspen+ simulations ending with warnings.
Aug 22, 2018
This is where that change happened in the old svn repo (login required). |
Bottom Up: SinterConsole.exe succeeds
|
AspenSinterConsumerConsole.exe fails
AspenSinterConsumerConsole.exe output
|
Turbine.Consumer.SimSinter: SinterConsumerRun.csSo I think the first step is to add an "if else WARNING" here and just proceed.
|
Handle Warnings and return as "success"Code committed and tested against the AspenPlus simulation provided. CCSI-Toolset/turb_sci_gate@00f2cef
|
New Download available here: |
ksbeattie
added a commit
that referenced
this issue
Aug 31, 2018
Pointing to the releases page and not just the binary to download. Part of #268
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
FOQUS used to be able to get simulation results for A+ simulations which ended in warnings. At some point this seems to have stopped working. Regardless of weather the simulation ends in a warning or convergence error the results should be passed back to FOQUS if at all possible.
I think I tracked down the problem. At some point lines were commented out that cause warnings to be treaded as unknown errors. Here is the location in the code https://github.com/CCSI-Toolset/turb_sci_gate/blob/master/Master/Turbine.Consumer.SimSinter/SinterConsumerRun.cs#L854.
I guess treating warnings as unknown errors may cause the results of the simulation not to be stored by Turbine? @boverhof can you check into this. If you need a test case @fsoepyan should be able to provide it.
The text was updated successfully, but these errors were encountered: