airboxlab / rllib-energyplus

Simple EnergyPlus environments for control optimization using reinforcement learning
MIT License
40 stars 8 forks source link

OutputControl setting overriden? #7

Closed sunchaesk closed 1 year ago

sunchaesk commented 1 year ago

In my IDF file, I have the OutputControl:Table:Style as CommaAndHtml, but for the folders generated by rllib for each of the episodes, doesn't contain the html files, and the csv file doesn't seem to be the default setting that I get when I run just plain E+ simulation.

I was reading through the code but couldn't see if there was a setting that overrides this, or if I am simply confused about some E+ settings.

Thanks.

antoine-galataud commented 1 year ago

Hi @sunchaesk,

In the code E+ is called with -r in order to generate the eplusout.csv file. See https://github.com/airboxlab/rllib-energyplus/blob/dd1f21a29b8e9aab73b2e0347df09e37e4670035/run.py#LL180C23-L180C27 Then you will need to update the IDF to output more variables or meters (using Output:Variable or Output:Meter). Note that you can request output variables using the python API (see https://nrel.github.io/EnergyPlus/api/python/datatransfer.html, request_variable method).

For the rest, there is no part of the code that changes the outputs, so it's safe to update the .idf directly.