NREL / OpenStudio

OpenStudio is a cross-platform collection of software tools to support whole building energy modeling using EnergyPlus and advanced daylight analysis using Radiance.
https://www.openstudio.net/
Other
494 stars 188 forks source link

Fix #5030 - C++ CLI: run.log is huge and missing the runner.registerWarning messages #5035

Closed jmarrec closed 9 months ago

jmarrec commented 9 months ago

Pull request overview

Pull Request Author

Labels:

Review Checklist

This will not be exhaustively relevant to every PR.

ci-commercialbuildings commented 9 months ago

CI Results for 87db4fb768f0dca378943a372b369c8c85a32670:

brianlball commented 9 months ago

yeah, it would also be good to do a quick study on the performance impact of logging. I've seen performance go to sh*t when there is too much logging going on, espically with runner.registerINFO etc