Translation warning in internal wall (airflow branch)

[kldjonge]

Any suggestions on how to get rid of these warnings? They occur during checking/translation and only when an internal wall is used in the branches with the embedded interzonal airflow implementation. When a lot of internal walls are present, it clouds the translation log.

They happen always and I use IDEAS.Examples.Tutorial.Example1 to check. I tried some things but without succes and would like some input/ideas.

[jelgerjansen]

When I use the latest master branch, I also get a warning for the area

Thus I checked the difference in area definition between the master branch and airflow branch. I noticed that in the latter branch, the model PartialSurface has an additional real expression AExp whose value is equal to A. The output of that block is then connected to propsBusInt instead of layMul.A.

I copied this approach for the variables for which a warning appears (v50 and use_custom_q50):

If you correctly define their value and connect them to propsBusInt, the warnings disappear. I think the reason for this is some sort of "decoupling" between propsBusInt (connected to zone 1) and propsBus_b (connected to zone 2), as the latter still has the default connections.

However, since I didn't model this implementation myself, I have no clue whether or not this reasoning makes sense. @kldjonge I don't know if the solution above might give you some inspiration to find the "origin" of the problem (and a more elegant solution to get rid of the warnings)? If not, maybe @Mathadon can help us out here?

[kldjonge]

Thanks Jelger! This is what was needed. In fact, the q50_zone model should be writing that signal but for some reason it doesn't recognize is as output/input. A simple routing through a realpassthrough and booleanpasstrough respectively solves it, just like you suggest.

[kldjonge]

I made these changes here: https://github.com/kldjonge/IDEAS/tree/Development_master togheter with some changes nececarry to avoid warnings about units since Dymola 2025

[kldjonge]

Beware: it also holds commits to avoid unit conversion warnings but these have been fixed in IBPSA

[jelgerjansen]

@kldjonge your fix is much more elegant, thanks! I'll merge this in the master branch once the interzonal airflow branch is merged.

Regarding the unit warnings: we merged the IBPSA library in the master branch at November 7 (commit 82957bb) thus this should be OK.