euniversal / umei-api-specification

UMEI API specifications
Apache License 2.0
3 stars 0 forks source link

Add the grid node id in the asset portfolio #12

Closed cdmNSIDE closed 2 years ago

cdmNSIDE commented 3 years ago

And remove the grid node id in the orders since it can be retrieved from the asset portfolio

narve commented 3 years ago

Hm this needs to be discussed. While a portfolio in general might have a grid node id, it is not obvious that it will always offer flexibility on that particular grid node. I don't see any particular use case for that now, but I will discuss it internally.

However, removing grid node id from orders does not make sense to us. If you post a BUY order you don't specify a portfolio, but you still want to declare where in the grid you want flexibility?

cdmNSIDE commented 3 years ago

Oh indeed, I forgot this particularity of the buy orders. I agree to keep the gridNodeId in the orders then. And in that case, there is no point to add it in the portfolio..

cdmNSIDE commented 3 years ago

If we do not add the gridNodeId in the portfolios, then I guess we may have to add a gridNodeId (or replace the assetPortfolioId by gridNodeId) in the baselines and in the measurements. What do you think ?

narve commented 3 years ago

We can not replace the portfolio id in a baseline with a grid node. The baselines applies to a specific asset(portfolio). There typically are many portfolios (and thus baselines) per grid node.

narve commented 3 years ago

Furthermore, the FSPs are not necessarily aware which grid node id the portfolio is placed on, that is the responsibility of the DSO.

narve commented 3 years ago

And in addition, we have a use case where we have movable assets, i.e. assets which can swap location (and thus grid node). The baselines would in that case better be linked to a portfolio, removing the need to update the grid node for already saved baselines.

However, we don't see a problem with adding grid node id to BaselineInterval per se. However, it should be up to the FMO whether this field is forbidden, optional or required.

ArnaudDebray commented 3 years ago

Hello Narve, I think there is a misunderstanding/misalignment on the workflow here. In our understanding, the asset portfolios are defined by the FSP/aggregator and should be related to a gridNode.

We should spend time during the meeting today to clarify the process. Then after good alignment on the process, we can discuss the impacts on the API.

Concerning the movable assets, can you clarify the use case ? From my business point of view, I see a lot of potential problems when doing this. And I really don't believe this has ever been considered in any of the demos.

narve commented 3 years ago

Note from meeting: Clean up the PR so that it only adds gridNodeId to Portfolio and nothing else.