Closed AntoineGautier closed 7 months ago
@JayHuLBL This PR goes along with https://github.com/lbl-srg/modelica-buildings/pull/3515 that has already been merged. Will you have a chance to review soon?
@AntoineGautier I was able to export the document. However, I'd firstly need to run npm run docker:build
from the server
package and then from client
folder run npm i
and npm start
.
Description
This includes necessary refactoring related to annotations newly added to MBL via https://github.com/lbl-srg/modelica-buildings/pull/3515. No functional changes.
MODELICA_BUILDINGS_COMMIT
now points to the latest commit of MBL master (6036d14).Related Issue(s)
This closes #357 and #365.
Testing
This has been tested locally
npm run test:ci
where only 1 test fails:Maybe due to a missing local dependency? Let's see what remote CI testing returns.
[EDIT on 10/6]
[x] There is an issue with the documentation export. The whole section about OA & RA damper control is missing. This needs to be understood and fixed before merging this PR.