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.

Sign up for GitHub

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

Jump to bottom

check for wrong derivative implementation #376

Closed
mwetter opened this issue Sep 12, 2020 · 0 comments · Fixed by #377
Closed

check for wrong derivative implementation #376

mwetter opened this issue Sep 12, 2020 · 0 comments · Fixed by #377
Assignees
@mwetter

Comments

@mwetter
Copy link
Member

mwetter commented Sep 12, 2020

Dymola 2021x beta2 issues warnings such as

Warning: function Buildings.Utilities.Psychrometrics.Functions.saturationPressureLiquid specified derivative Buildings.Utilities.Psychrometrics.Functions.BaseClasses.der_saturationPressureLiquid, but
argument TSat function did not match argument Tsat of derivative.

This issue is to catch these warnings and fail the unit tests.
@mwetter mwetter self-assigned this Sep 12, 2020
@mwetter mwetter changed the title check for wrong derivative annotation check for wrong derivative implementation Sep 12, 2020
mwetter added a commit that referenced this issue Sep 12, 2020
@mwetter
Added test for derivative specification
bf7de10 
For #376
@mwetter mwetter mentioned this issue Sep 12, 2020
Merged
mwetter added a commit to lbl-srg/modelica-buildings that referenced this issue Sep 12, 2020
@mwetter
Merged IBPSA, lbl-srg/BuildingsPy#376
f02747c
@mwetter mwetter mentioned this issue Sep 12, 2020
Merged
mwetter added a commit that referenced this issue Sep 13, 2020
@mwetter
Added test for derivative specification (#377)
c921ec4 
* Added test for derivative specification

For #376

* Added new line

* Corrected number of keys and messages
sjoelund pushed a commit to OpenModelica/OMLibraries that referenced this issue Sep 14, 2020
@OpenModelica-Hudson
Bump libraries
2ee8334 
- https://github.com/ibpsa/modelica.git branch master updated to ff8ec1e2cccd08c5cacfe223a87d407d9ad868f5.
  * [ibpsa/modelica-ibpsa@ff8ec1e ff8ec1e] Corrected units and html syntax
  * [ibpsa/modelica-ibpsa@a80764b a80764b] Merge branch 'master' into issue1393_derivativePsychrometrics
  * [ibpsa/modelica-ibpsa@67a66b7 67a66b7] Corrected argument names

- https://github.com/lbl-srg/modelica-buildings.git branch master updated to 61cfd3362684e4052319b4c40c6d0660561c6980.
  * [lbl-srg/modelica-buildings@61cfd33 61cfd33] Merge pull request #2135 from lbl-srg/IBPSASync
  * [lbl-srg/modelica-buildings@7bac9d0 7bac9d0] Corrected units and html syntax
  * [lbl-srg/modelica-buildings@f02747c f02747c] Merged IBPSA, lbl-srg/BuildingsPy#376

- https://github.com/SolarTherm/SolarTherm.git branch master updated to 0dc31516f857ebf0e0ac49c7934887d9bfbcbed1.
  * [SolarTherm/SolarTherm@0dc3151 0dc3151] get oelt and A_land for a known field
  * [SolarTherm/SolarTherm@9490b05 9490b05] add an option to simulate oelt from a known field
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwetter mwetter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added test for derivative specification lbl-srg/BuildingsPy
1 participant
@mwetter