-
Notifications
You must be signed in to change notification settings - Fork 7
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
Error when running simulation batch #1702
Milestone
Comments
rwmcintosh
added a commit
that referenced
this issue
Sep 23, 2022
rwmcintosh
added a commit
that referenced
this issue
Sep 23, 2022
rwmcintosh
added a commit
that referenced
this issue
Sep 26, 2022
not sure if the issue was fixed. @rwmcintosh With ospsuite-R 11.1.115 I don't get any error, but the results are empty:
|
@Yuri05 with the latest version the error is thrown. Probably the Core dlls were not updated. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@PavelBal commented on Fri Jul 15 2022
Vehicle.zip
Probably an errorr in core.
When seeting all molecules as variable molecules in the attached simulation, runnning the batch produces the error that some molecules do not exist in the simulation.
Error:
@PavelBal commented on Fri Jul 15 2022
@abdelr Could you look into it when you have time? I can help you translating the example to .NET test if necessary.
@PavelBal commented on Thu Sep 22 2022
@rwmcintosh
@rwmcintosh commented on Thu Sep 22 2022
OSPSuite.Core/src/OSPSuite.Core/Domain/Services/SimModelManagerBase.cs
Line 84 in adf6f9f
For reference.
In the attached project there is a discrepancy at this line where the two lists are not perfectly 1 to 1.
Here is the list of paths in variableMoleculePaths that is not available from SpeciesProperties. Since this list of variableMoleculePaths comes from the simulation, I thought SpeciesProperties would contain them all.
The paths available in SpeciesProperties that are not in variableMoleculePaths are
From the R code, the variableMoleculePaths are retrieved by
@Yuri05 @msevestre Any idea why these two lists are not matching?
@rwmcintosh commented on Thu Sep 22 2022
See my unit test which translates the R code
https://github.com/rwmcintosh/OSPSuite.Core/blob/3b152081707894657df5449668bf15021093c4a4/tests/OSPSuite.R.Tests/Services/ConcurrentSimulationRunnerSpecs.cs#L56
@rwmcintosh commented on Fri Sep 23 2022
After discussion with @msevestre and @Yuri05 today, we decided that the bug itself can be fixed meaning - if the list of variable molecules is not a precise match we will only try to vary the intersection of the lists. The issue is caused because SimModel optimizes the molecules that won't be varied into parameters. The list in this example is created before the SimModel initialization, so there is no way for the application to know this will happen.
To enable the steady state use case, some more reorganization will be required in SimModel and OSPSuite.Core so that the list of variable molecules and variable parameters are actually known by the caller.
@rwmcintosh commented on Fri Sep 23 2022
Will have to revisit when SimModel supports the fix.
The text was updated successfully, but these errors were encountered: