Skip to content

Cleanup IDD issues with object-list and reference tags#4597

Merged
mjwitte merged 19 commits intodevelopfrom
4525-Object-list_Referencing
Nov 25, 2014
Merged

Cleanup IDD issues with object-list and reference tags#4597
mjwitte merged 19 commits intodevelopfrom
4525-Object-list_Referencing

Conversation

@Myoldmopar
Copy link
Member

@mmAtTs If you check out the test failures at the (currently) bottom of this pull request, you'll see there are some error messages coming out. If you could address those, it would be nice to get this in.

@mjwitte
Copy link
Contributor

mjwitte commented Nov 19, 2014

@Myoldmopar @mmAtTs After the errors are fixed, I still need to do another review. Been busy with other tasks this week.

@mitchute
Copy link
Collaborator

@Myoldmopar @mjwitte There was one error that I introduced in the UnitarySystemPerformance:HeatPump:Multispeed object which I corrected here, 2121b60. I ran the test suite on the files in question and am seeing no diffs on my machine. Let's see what CI comes back with.

@mitchute
Copy link
Collaborator

@Myoldmopar CI is showing what looks to me to be minor diffs in the eio files for DElightCFSWindow.idf. I am not able to replicate it on my machine. Any ideas what it might be or how to track it down?

@Myoldmopar
Copy link
Member Author

As far as I can tell, they are absolute random diffs from the DeLight library. Presumably there is something uninitialized or random going on over there, but we haven't yet nailed it down. @EnergyArchmage mentioned that seeing random diffs with DeLight was nothing new, so we're just moving forward, hopeful to figure out the problem sometime soon. So, don't worry about it, it's nothing your branch did.

@mjwitte
Copy link
Contributor

mjwitte commented Nov 24, 2014

@mmAtTs @Myoldmopar Fixed a couple of errors, quibbled Non-Dimensional back to Nondimensional, and some other cleanup. Please review to see if you agree. @mmAtTs This turned out to be much large than anticipated - thanks for sticking with this.

mjwitte added a commit that referenced this pull request Nov 25, 2014
@mjwitte mjwitte merged commit 5ab080e into develop Nov 25, 2014
@mjwitte mjwitte deleted the 4525-Object-list_Referencing branch November 25, 2014 14:32
@mjwitte
Copy link
Contributor

mjwitte commented Nov 25, 2014

Fixes #4525

mjwitte added a commit that referenced this pull request Dec 9, 2014
… MJW change from part of #4597) SurfaceNames are only base surfaces
@Myoldmopar Myoldmopar changed the title 4525 object list referencing Cleanup IDD issues with object-list and reference tags Mar 7, 2015
@Myoldmopar Myoldmopar added the Defect Includes code to repair a defect in EnergyPlus label Mar 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Defect Includes code to repair a defect in EnergyPlus

5 participants