ElectricalEnergyStorage icon indicating copy to clipboard operation
ElectricalEnergyStorage copied to clipboard

Missing fixed=false in Modelica.Electrical.EnergyStorages.Batteries.Components.Calculators.Cycles

Open modelica-trac-importer opened this issue 8 years ago • 4 comments

Reported by pharman on 15 Oct 2012 16:27 UTC In Modelica.Electrical.EnergyStorages.Batteries.Components.Calculators.Cycles the parameter integrator.y_start is set at initialisation, but doesn't have the attribute fixed=false set so as a parameter defaults to fixed=true.

Therefore the declaration:

  Modelica.Blocks.Continuous.Integrator integrator(y_start=cyclesini);

should be changed to:

  Modelica.Blocks.Continuous.Integrator integrator(y_start(fixed=false)=cyclesini);

Migrated-From: https://trac.modelica.org/Modelica/ticket/878

modelica-trac-importer avatar Jan 30 '17 19:01 modelica-trac-importer

Comment by dietmarw on 4 Dec 2012 16:34 UTC Library is not going to be part of MSL 3.2.1 so rescheduling for possible inclusion in MSL 3.3

modelica-trac-importer avatar Jan 30 '17 19:01 modelica-trac-importer

Modified by beutlich on 18 Jun 2014 09:33 UTC

modelica-trac-importer avatar Jan 30 '17 19:01 modelica-trac-importer

The pendantic model check of Dymola rises an error message for

Modelica.Blocks.Continuous.Integrator integrator(y_start(fixed=false)=cyclesini);

However, I do not quite understand the intention of your proposal. Do you want to explicitly overwrite the integrator initial condition?

christiankral avatar Feb 08 '17 11:02 christiankral

The library structure is designed for quite sophisticated applications. Therefore, the library should definitely be re-designed and simplified before integrating it into the MSL.

christiankral avatar Feb 08 '17 11:02 christiankral