id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
5385	-d=evaluateAllParameters should not evaluate non-fixed parameters	Francesco Casella	Per Östlund	"Please consider the following simple test case:
{{{
model TestParameters
  parameter Real p(fixed = false);
  parameter Real q = 2*p;
initial equation
  p - 2 = 0;
end TestParameters;
}}}
The NF handles this nicely in the nightly build corresponding to [https://github.com/OpenModelica/OMCompiler/commit/04f106144e229956b4351723fd8bbbe06c146f35 04f1061]. One of the three commits on March 11 (I'm not sure which one) apparently broke something, so that the nightly corresponding to [https://github.com/OpenModelica/OMCompiler/commit/965626166cf456d172df14cd97467d52f1bd313a 9656261] gives the following error:
{{{
[1] 11:58:17 Translation Notification
[Test: 2:3-2:34]: From here:

[2] 11:58:17 Translation Error
[Test: 3:3-3:25]: Constant p is used without having been given a value.
}}}
This test model is perfectly fine, since the parameters without binding equations but having {{{fixed = false}}} are matched to initial equations. This case is apparently not covered in the testuite, so it didn't cause any obvious regression, but it happens to break a number of models at Dynamica, that would otherwise happily run with the new frontend, and don't run with the old frontend on the other hand.

Could you please fix this ASAP and add the test case to the testsuite?

Thanks!"	defect	closed	critical	1.14.0	New Instantiation		fixed		Andrea Bartolini