#5167 closed defect (fixed)
Issue with package constants in HelmholtzMedia with the NF
| Reported by: | Francesco Casella | Owned by: | Per Östlund |
|---|---|---|---|
| Priority: | high | Milestone: | 2.0.0 |
| Component: | New Instantiation | Version: | |
| Keywords: | Cc: | m.thorade@… |
Description
Please check HelmholtzMedia.Examples.ConvergenceTest.Ancillary_Saturation. The following error is reported:
HelmholtzMedia_HelmholtzMedia.Examples.ConvergenceTest.Ancillary_Saturation_functions.c:2680:10:
error: use of undeclared identifier '_Medium'
tmp2 = _Medium._fluidConstants[1]._criticalMolarVolume;
^
Change History (6)
comment:1 by , 6 years ago
| Cc: | added |
|---|
comment:2 by , 6 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
follow-up: 4 comment:3 by , 6 years ago
This error seems to be similar!?
HelmholtzMedia_HelmholtzMedia.Examples.Parameter.State_pT_parameter_Transport_functions.c:15912:39: error: use of undeclared identifier '_Medium' tmp2 = size_of_dimension_base_array(_Medium._dynamicViscosityCoefficients._a, ((modelica_integer) 1));
comment:4 by , 6 years ago
Replying to Matthis Thorade <matthis.thorade@…>:
This error seems to be similar!?
HelmholtzMedia_HelmholtzMedia.Examples.Parameter.State_pT_parameter_Transport_functions.c:15912:39: error: use of undeclared identifier '_Medium' tmp2 = size_of_dimension_base_array(_Medium._dynamicViscosityCoefficients._a, ((modelica_integer) 1));
Fixed in 00ef403a.
follow-up: 6 comment:5 by , 6 years ago
Nice!
The following two also have a similar error message. Again, I don't know if that means they are related at all...
HelmholtzMedia_HelmholtzMedia.Examples.Parameter.setSat_parameter.c:80:3: error: use of undeclared identifier '$Psat_p$Pliq' $Psat_p$Pliq = tmp2._liq; ^
https://libraries.openmodelica.org/branches/newInst/HelmholtzMedia/files/HelmholtzMedia_HelmholtzMedia.Examples.Parameter.setSat_parameter.err
https://libraries.openmodelica.org/branches/newInst/HelmholtzMedia/files/HelmholtzMedia_HelmholtzMedia.Examples.ConvergenceTest.setSat.err
comment:6 by , 6 years ago
Replying to Matthis Thorade <matthis.thorade@…>:
Nice!
The following two also have a similar error message. Again, I don't know if that means they are related at all...
It might look similar, but it's a completely different issue. I opened #5225 for it.
Fixed in 02ecfb7.