Update simulator callback to process action, set up the same as observation (callback_end_zone_timestep_after_zone_reporting)
Motivation and Context
This callback is better suited for action processing and doesn't interfere with observation processing like others do. Reproducibility of experiments using seeds remains available, except in rare cases where control begins at the first step instead of the second. It seems to be an issue with the EnergyPlus API and currently, nothing can be done to prevent it. The topic is also discussed in #368.
Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Description
Update simulator callback to process action, set up the same as observation (
callback_end_zone_timestep_after_zone_reporting
)Motivation and Context
This callback is better suited for action processing and doesn't interfere with observation processing like others do. Reproducibility of experiments using seeds remains available, except in rare cases where control begins at the first step instead of the second. It seems to be an issue with the EnergyPlus API and currently, nothing can be done to prevent it. The topic is also discussed in #368.
Types of changes
Checklist:
autopep8
second level aggressive.isort
.cd docs && make spelling && make html
pass (required if documentation has been updated.)pytest tests/ -vv
pass. (required).pytype -d import-error sinergym/
pass. (required)Changelog:
callback_end_zone_timestep_after_zone_reporting
)