Opened 9 years ago

Closed 4 years ago

#3761 closed defect (wontfix)

Buildings coverage went down after library update

Reported by: Adrian Pop Owned by: Adrian Pop
Priority: high Milestone: Future
Component: Frontend Version: v1.10.0-dev-nightly
Keywords: Cc: Michael Wetter

Description

36 new failures:
https://test.openmodelica.org/hudson/job/Buildings_Flattening/922/
Most of the models fail with:

Buildings latest/Fluid/Movers/BaseClasses/PartialFlowMachine.mo:185:3-189:70:writable] 
Error: Variable Types.PrescribedVariable.Speed not found in scope Buildings.Fluid.Movers.SpeedControlled_y$fan32.

Change History (5)

comment:1 by Adrian Pop, 9 years ago

Owner: changed from somebody to Adrian Pop
Status: newaccepted

comment:2 by Martin Sjölund, 9 years ago

Cc: Michael Wetter added

comment:3 by Michael Wetter, 9 years ago

Thanks for reporting. This is now fixed through https://github.com/lbl-srg/modelica-buildings/pull/496

Note in particular the commit
https://github.com/iea-annex60/modelica-annex60/commit/02c325b7166c5280303c579e35c3418d4b46ba8a
(introducing an intermediate variable eliminated the error Types.PrescribedVariable.Speed not found)

Also
https://github.com/iea-annex60/modelica-annex60/commit/a583143244de3604ff613a272c0518920fc89056
and
https://github.com/iea-annex60/modelica-annex60/commit/88cff476736f36bdd7948849aae74ca7f0963f67
indicates a problem with the front end.

All of these commits are now also on the master of the Buildings library, and under review for the Annex60 master.

comment:4 by Adrian Pop, 9 years ago

Thanks mwetter but we should fix the issues anyway.

comment:5 by Francesco Casella, 4 years ago

Resolution: wontfix
Status: acceptedclosed

Probably obsolete at this point

Note: See TracTickets for help on using tickets.