search

ladybug-tools / honeybee

:honeybee: A python library to create, run and visualize radiance studies. Core library of Honeybee[+].
http://ladybug-tools.github.io/honeybee/docs
GNU General Public License v3.0
93 stars 25 forks source link

output variables are missing on the readEPResult for result file generated by the runEnergySimulation component #132

Closed hopezh closed 7 years ago

hopezh commented 7 years ago

It seems that several key output variables, such as cooling loads, are missing on readEPResult for result file generated by the runEnergySimulation , and this is not an issue when using the exportToOpenStudio.

chriswmackey commented 7 years ago

@hopezh ,

What version of EnergyPlus are you using? I think the most recent version of E+ has had some formatting changes that would cause certain outputs to not be read. The most recent version of OpenStudio includes automated ways to correct for this, though. Maybe we are truly are nearing the end of the line for the E+ component as it's getting difficult to support two components that do the same thing.

-Chris

hopezh commented 7 years ago

Dear Chris,

It seems there are two "\n" new line characters accidentally missing from line 2314 andd 2315 in the code for the current runEergySimulation component. This causes the "zone ideal loads supply air total cooling energy" variable tangled with other output variables in one line in the idf file.

I've put the observations in this post:

Hope you can kindly help to verify. If this is the case, appreciate if you can kindly update the runEP component as we still need to use its PV related functions for the time being before this function is added to the run OpenStudio component.

Thanks. -Ji

chriswmackey commented 7 years ago

@hopezh , Thanks for the clarification. This doesn't seem to be the result of the new E+ but rather something that happened as a result of some people wanting quick access to the HTML output from E+. I'll try to fix this later today. It should be quick.

chriswmackey commented 7 years ago

@hopezh , Sorry for getting back to this so late. Things were very busy 2 weeks ago. I just added in the correction that you suggested and everything is working now: https://github.com/mostaphaRoudsari/honeybee/commit/90279c3e1bb7247e7469b4fb0bc49957e83d0d32 Thanks again for letting us know about this, -Chris

hopezh commented 7 years ago

Dear Chris, thank you very much for the quick update. Do take care under such a heavy workload.