Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CompoundEnergyAsset #14

Open
jpwijbenga opened this issue Feb 17, 2021 · 1 comment
Open

Add CompoundEnergyAsset #14

jpwijbenga opened this issue Feb 17, 2021 · 1 comment

Comments

@jpwijbenga
Copy link
Contributor

For the EYE simulator we regularly use Compound Assets to represent Assets that have some sub-assets, but that should behave integrated as one actor on the market place. Eg. Electrolyzer + Battery, Electrolyzer + HydrogenStore, Wind park + Battery.

These are always Energy Assets and we implement helper methods in Energy Asset Mixin.
Our CompoundAsset should be able to use these methods and currently is not.
So we need a CompoundEnergyAsset that is a subtype of EnergyAsset. Can this be added to ESDL?

@ewoudwerkman
Copy link
Member

CompoundAssets are just there so you can group assets together. This was mainly introduced for heat grids that have similar requirements as EYE, e.g. a producer together with a pump, a PID controller and a pressure sensor. Its just grouping of assets.

We did not create an CompoundEnergyAsset (CEA), because EnergyAssets have ports and then the connections to these ports would become ambigious. E.g. what to do with a connection to the InPort of a CAE? to what is it connected inside the compound and how, or is it connected at all? What happens if the CEA's ports are not used, but the ones of the assets inside the compound are? It would be difficult to enforce rules on these ports.
Similar discussions for other features of assets such as behaviour and control strategies.

I suggest to use the CompoundAsset mixin as a facade for the energy assets it contains and factor out the generic methods that are shared between the CompoundAsset and the EnergyAsset (e.g. add this to the shared super class such as Asset?) Does that work for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants