rdnfn / beobench

A toolkit providing easy and unified access to building control environments for reinforcement learning (RL).
https://beobench.readthedocs.io
MIT License
37 stars 4 forks source link

Allow empty env.config in config yaml file #90

Closed rdnfn closed 2 years ago

rdnfn commented 2 years ago

Problem

There can be issues with setting env.config to None/null in a config file or leaving out this setting entirely. This is because some parts of the scheduler assume a non-empty env.config. This issue was originally raised in #82 by @HYDesmondLiu.

Potential Solution

There are multiple ways to achieve this:

  1. automatically add env config if empty (requires access to this in code).
  2. remove call to config outside env_creator script.
rdnfn commented 2 years ago

On the current dev version, the name attribute is no longer required anywhere in Beobench scheduler or agent code. Thus, this issue will be resolved in the next release.