search

INTO-CPS-Association / hybrid-cosim-framework

A native framework for generated hybrid co-simulations
GNU General Public License v3.0
0 stars 0 forks source link

Incorrect comment #1

Open CThuleHansen opened 6 years ago

CThuleHansen commented 6 years ago

The following comment within ExecuteInRules is incorrect, as the function is not invoked on every DoStep. https://github.com/INTO-CPS-Association/hybrid-cosim-framework/blob/1545c9493a98e678a8c9050dfea7e069e5c7d013/semantic-adaptation/src/SemanticAdaptation.h#L212-L218

The function is only executed on every SetInput on the external FMU https://github.com/INTO-CPS-Association/hybrid-cosim-framework/blob/1545c9493a98e678a8c9050dfea7e069e5c7d013/semantic_adaptation_samples/CtrlMultiRate/src-gen/Fmu.cpp#L219

The enabledInRules should still be cleared though.

However, FlushAllEnabledInRules is executed on every DoStep https://github.com/INTO-CPS-Association/hybrid-cosim-framework/blob/1545c9493a98e678a8c9050dfea7e069e5c7d013/semantic-adaptation/src/SemanticAdaptation.h#L257-L265

Is the behaviour or the comment correct?