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

Merged IBPSA, issue1556_filePermissions and issue1109_wetCoilEpsNTU #2780

Merged
merged 15 commits into from
Dec 2, 2021

Conversation

mwetter
Copy link
Member

@mwetter mwetter commented Nov 24, 2021

This merges IBPSA issue1556_filePermissions and issue1109_wetCoilEpsNTU.
(Note that issue1556_filePermissions is about to be merged through #2779)

@AntoineGautier : Can you please make sure the changes from issue1109_wetCoilEpsNTU are correct. The changes are from ibpsa/modelica-ibpsa#1549 which diverged from what is on the master of the Buildings library.

Copy link
Contributor

@AntoineGautier AntoineGautier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a few inline comments to be discussed with you. They may require changes.
I also realized that the changes from PR#2711 were not taken into account in the IBPSA branch. I reintroduced them.

@mwetter mwetter merged commit 7255b26 into master Dec 2, 2021
@mwetter mwetter deleted the IBPSASync_issue1109_wetCoilEpsNTU branch December 2, 2021 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants