-
Notifications
You must be signed in to change notification settings - Fork 48
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
Make FMU parameters tunable when importing #85
Comments
Documentation about tunable parameters: |
You have to ZIP them or append |
It should be possible to expose tunable parameters as S-Function Run-Time Parameters. |
Any news on this topic? We really need this. |
Any update? |
I've created a prototype (443d57f) based on Pros:
Cons:
|
Another (and probably more robust) approach would be to simply add the tunable parameters as discrete inputs, leveraging the existing infrastructure. |
Would you expose all tunable parameters as discrete inputs or just a few selected by the user? The downside of this approach is that you have to create lots of "constant" blocks and connect them to the right inputs. This might also have a performance impact. Parameters are only written when changed, inputs every timestep. |
No. These inputs would have to be added individually, like the output ports.
We could keep track of the previous value and only set it, when it has changed. |
This would work nicely for my use case. |
@t-sommer I want to test the new tunable parameters with the 3.0 alpha. What do I need to do to make a parameter tunable? Do I have to select it as an input? I noticed that not all parameters have the input option available. Would you mind commenting on what is required for a parameter for it to be available to be defined as a tunable parameter? |
Only tunable parameters can be exposed as block inputs. |
With the FMIKit FMU importer, parameters of the FMU can only be changed at the beginning of the simulation. With the Mathworks FMU importer (tested with 2019b), the FMU parameters are tunable and can be changed during runtime.
The text was updated successfully, but these errors were encountered: