Closed mjwitte closed 8 months ago
OK, so my changes over in #6983 cause this to be a fatal error. Here's the progression:
I'm changing this milestone to E+ next and moving on.
@Myoldmopar Seems like we should just make it required in IDD. It's a simple IDD change, and the user will know where to go look, no?
For Output:Table:SummaryReports
, the behavior has been.
v8.7, v8.8: ** Severe ** Output:Table:SummaryReports Field[2]="", invalid report name -- will not be reported.
Simulation continues to completion.
v8.9: unexplained fatal error.
v9.0: ** Fatal ** Blank report name in Oputput:Table:SummaryReports
. This should have included a transition rule to clean out any blanks so that working files from v8.8 and before would transition successfully. Now these will fail if there are any lurking out there with a blank report field.
Of course we also have a bunch of other old key choice synonyms out there that were never added to transition, so add this to #6601
Just retested this, behavior is still the same as noted here. Forgot to deal with the blank field case in #6601, so will make a new pull request to address this in Transition-V8-8-0-to-V8-9-0.
Just FYI this seems to still be an issue in 9.6, at least for the Output:Table:Annual. It's easy enough to fix in the text file, but the IDFeditor causes the fatal error. There is also a typo in the error message where Output is spelled 'Oputput'
Reopening this issue. Seems we should be able to trap blank unrequired fields and not error out on them, maybe.
@mjwitte and @Myoldmopar I was thinking about taking on this issue. It seems like this issue started out just trying to make sure a blank report name in Output:Table:SummaryReport occurs. This would be a straightforward fix. But near the end of the conversation, it seems like maybe this is an issue in general about how blank unrequired fields should be treated. That seems very open-ended to try to fix since it probably matters for each field. So which is it?
This only applies to blank fields in a few objects that get pre-processed by the input processor. Other blank fields return blank or zero or their default.
@Myoldmopar 's changes in 6983 let blank fields get past the input processor for the pertinent report objects. So, if I'm following correctly, all that's left to do here is to change the fatals added in #6983 to warnings for Output:Table:Monthly, Output:Table:Annual, and Oputput:Table:SummaryReports.
That is great that it is only those couple of input objects. Thanks @mjwitte
Do you think they should be Severes or Warnings? I was leaning toward Warnings.
Warnings.
Issue overview
This object
results in this error file
Help desk ticket 13045 has blank meter names in Output:Table:Monthly and Output:Table:Annual
Output:Table:Monthly, Overall HVAC System Energy, !- Name 3, !- Digits After Decimal Cooling:Electricity, !- Variable or Meter 1 Name SumOrAverage, !- Aggregation Type for Variable or Meter 1 , !- Variable or Meter 2 Name SumOrAverage; !- Aggregation Type for Variable or Meter 2
Output:Table:Annual, OUTPUT TABLE 1, !- Name , !- Filter , !- Schedule Name , !- Variable or Meter or EMS Variable or Field 1 Name , !- Aggregation Type for Variable or Meter 1 2, !- Digits After Decimal 1 , !- Variable or Meter or EMS Variable or Field 2 Name , !- Aggregation Type for Variable or Meter 2 2, !- Digits After Decimal 2 , !- Variable or Meter or EMS Variable or Field 3 Name , !- Aggregation Type for Variable or Meter 3 2, !- Digits After Decimal 3 , !- Variable or Meter or EMS Variable or Field 4 Name , !- Aggregation Type for Variable or Meter 4 2, !- Digits After Decimal 4 , !- Variable or Meter or EMS Variable or Field 5 Name , !- Aggregation Type for Variable or Meter 5 2, !- Digits After Decimal 5 , !- Variable or Meter or EMS Variable or Field 6 Name , !- Aggregation Type for Variable or Meter 6 2, !- Digits After Decimal 6 , !- Variable or Meter or EMS Variable or Field 7 Name , !- Aggregation Type for Variable or Meter 7 2, !- Digits After Decimal 7 , !- Variable or Meter or EMS Variable or Field 8 Name , !- Aggregation Type for Variable or Meter 8 2, !- Digits After Decimal 8 , !- Variable or Meter or EMS Variable or Field 9 Name , !- Aggregation Type for Variable or Meter 9 2, !- Digits After Decimal 9 , !- Variable or Meter or EMS Variable or Field 10 Name , !- Aggregation Type for Variable or Meter 10 2, !- Digits After Decimal 10 , !- Variable or Meter or EMS Variable or Field 11 Name , !- Aggregation Type for Variable or Meter 11 2, !- Digits After Decimal 11 , !- Variable or Meter or EMS Variable or Field 12 Name , !- Aggregation Type for Variable or Meter 12 2, !- Digits After Decimal 12 , !- Variable or Meter or EMS Variable or Field 13 Name , !- Aggregation Type for Variable or Meter 13 2, !- Digits After Decimal 13 , !- Variable or Meter or EMS Variable or Field 14 Name , !- Aggregation Type for Variable or Meter 14 2, !- Digits After Decimal 14 , !- Variable or Meter or EMS Variable or Field 15 Name , !- Aggregation Type for Variable or Meter 15 2; !- Digits After Decimal 15 Error file results:
Program Version,EnergyPlus, Version 9.0.0-63db0a5a1f, YMD=2018.08.24 09:46, Warning: Node connection errors not checked - most system input has not been read (see previous warning). Fatal error -- final processing. Program exited before simulations began. See previous error messages. EnergyPlus Warmup Error Summary. During Warmup: 0 Warning; 0 Severe Errors. EnergyPlus Sizing Error Summary. During Sizing: 0 Warning; 0 Severe Errors. ***** EnergyPlus Terminated--Fatal Error Detected. 0 Warning; 0 Severe Errors; Elapsed Time=00hr 00min 11.61sec
Details
Some additional details for this issue (if relevant):
Checklist
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
[x] Defect file attached here 6919-1ZoneUncontrolled-v89.idf.txt
[ ] Ticket added to Pivotal for defect (development team task)
[ ] Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)