Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Specific internal energy in PartialSimpleIdealGasMedium #600

Closed
modelica-trac-importer opened this issue Jan 14, 2017 · 4 comments
Closed
Assignees
Labels
bug Critical/severe issue L: Media Issue addresses Modelica.Media P: high High priority issue
Milestone

Comments

@modelica-trac-importer
Copy link

Reported by msielemann on 31 Aug 2011 09:02 UTC
In 8a5a9ff, a modification was made to function specificInternalEnergy of PartialSimpleIdealGasMedium.

redeclare function extends specificInternalEnergy 
  "Return specific internal energy"
  extends Modelica.Icons.Function;
algorithm 
  // u := (cp_const-R_gas)*(state.T-T0);
  u := cp_const*(state.T-T0) - R_gas*state.T;
end specificInternalEnergy;

The change introduces an inconsistency between the offset in specific enthalpy and that in specific internal energy and should be undone.


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

@modelica-trac-importer modelica-trac-importer added this to the Design74 milestone Jan 14, 2017
@modelica-trac-importer modelica-trac-importer added bug Critical/severe issue L: Media Issue addresses Modelica.Media P: high High priority issue labels Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Modified by msielemann on 31 Aug 2011 09:03 UTC

@modelica-trac-importer
Copy link
Author

Comment by dietmarw on 1 Mar 2012 14:47 UTC
Next MSL release seems to be a more fitting milestone for this.

@modelica-trac-importer modelica-trac-importer modified the milestones: MSL3.2.1, Design74 Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Modified by hubertus on 28 May 2012 22:50 UTC

@modelica-trac-importer
Copy link
Author

Comment by hubertus on 10 Jun 2012 23:26 UTC
This has also been fixed in the merge of a251c81

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Critical/severe issue L: Media Issue addresses Modelica.Media P: high High priority issue
Projects
None yet
Development

No branches or pull requests

2 participants