Changes between Initial Version and Version 1 of Ticket #3102, comment 19


Ignore:
Timestamp:
2015-02-25T11:23:44Z (10 years ago)
Author:
Francesco Casella

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #3102, comment 19

    initial v1  
    11The original motivation of this ticket was that test cases using the Modelica.Media IF97 water model were unnecessarily slow because of repeated calls to the computationally-intensive waterBaseProp_ph function with the same arguments. If CSE is implemented correctly, this should never be the case.
    22
    3 Issue 1: in the test model by Martin (comment:15), the expression sin(time) shows up twice. I would expect it to be only computed once in an auxiliary variable, which is then used twice. What I see, instead, is two CSE auxiliaries, which is not useful at all.
     3Issue 1: in the test model by Martin (comment:15), the expression sin(time) shows up twice. I would expect it to be only computed once in an auxiliary variable, which is then used twice. What I see, instead, is two CSE auxiliaries each computing sin(time), which is not useful at all.
    44
    5 Issue 2: as of r24759, code generated for the {{{ThermoPower.Examples.CISE.CISESim120501}}} model still contain duplicated calls, e.g.:
     5Issue 2: as of r24759, code generated for the {{{ThermoPower.Examples.CISE.CISESim120501}}} model still contains duplicated calls, e.g.:
    66{{{
    77$CSE75 := Modelica.Media.Water.IF97_Utilities.waterBaseProp_ph(