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
505 stars 192 forks source link

PAT fails mysteriously on Standard Reports or Calibration Reports with no errors #1064

Closed asparke2 closed 10 years ago

asparke2 commented 10 years ago

This one has hit several users, and is particularly bad because it is very difficult to find the cause.

To replicate this bug:

  1. Create a new PAT project, and select a baseline model.
  2. Run. It should succeed, assuming you have a weather file and running baseline model.
  3. Open the Online BCL Dialog
  4. Under Reporting>QAQC, download "Standard Reports" and "Calibration Reports"
  5. In PAT, click "Update BCL Measures." It should say 2 measures updated.
  6. Run. It should fail.

The issue is that OpenStudio is hard-coded to expect the default Standard Reports and Calibration Reports Measures to live in scripts/StandardReports and scripts/CalibrationReports respectively. See the params.json file excerpt below. When it does not find the files in these locations, it fails, but does not report out the reason.

"param" :       {
 "children" : [
  {
   "param" :          {
    "value" : "resources/report.html.in"
   },
   "param_index" : 0
  }
 ],
 "value" : "C:/Projects/Audit Tool/SiteDataSimuwattDemo/Tyndall/1060/1060 ECMs/scripts/StandardReports/resources/report.html.in"
},

Since the same Standard Reports and Calibration Reports Measures that are default in PAT/OS App are also uploaded on BCL, if you happen to download those Measures from BCL and click "Update BCL Measures," the directory names get changed to the UIDs that are used for all other BCL Measures.

Here is what the scripts directory looks like before the update: goodscriptsdir

Here is what it looks like after the update: badscriptsdir

These are the same 2 Measures, but with different folder names.

Possible fixes: Make the UID of the installed Measures different than the BCL Measures Hard code PAT and OS App to look for the UIDs instead of "StandardMeasures" There are probably other that I haven't thought of

axelstudios commented 10 years ago

This appears to be the same as #1046