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

Issue 46 show screenshots in documentation #60

Merged
merged 7 commits into from Mar 19, 2021

Conversation

nhoening
Copy link
Contributor

@nhoening nhoening commented Mar 11, 2021

  • incorporate screenshots from SeitaBV/screenshots
  • add some missing content
  • talk about flexibility opportunities rather than flexibility actions
  • remove two old pics

nhoening added 2 commits Mar 11, 2021
…ots higher up, talking about opportunities rather than actions and that certain functionality visualising them is only mocked; delete components page and images
@nhoening nhoening requested a review from Flix6x Mar 11, 2021
@nhoening nhoening linked an issue Mar 11, 2021 that may be closed by this pull request
Flix6x
Flix6x approved these changes Mar 19, 2021
Copy link
Contributor

@Flix6x Flix6x left a comment

  • Great to see the screenshots coming out of a separate repo.
  • The terminology around flexibility opportunities and activation requires more discussion, but considering that we take out the control page for now, we can tackle that later.
  • Some smaller reformulation requested.

documentation/views/dashboard.rst Show resolved Hide resolved
@@ -1,34 +1,33 @@
.. _control:

*****************
Flexibility actions
Flexibility opprtunities
Copy link
Contributor

@Flix6x Flix6x Mar 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of underdefined/ambiguous terminology in this section. I propose we discuss this thoroughly in case we activate this UI view.

Copy link
Contributor Author

@nhoening nhoening Mar 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note: this page is currently visible in the documentation. I suggest to move the content which does not talk about the UI into a new page which deals with concepts. That PR for #45 will land soon in your queue. There we might give it another look.

documentation/views/portfolio.rst Outdated Show resolved Hide resolved
Costs and revenues are calculated based on the internal method for profit sharing.
Asset owners that provide flexibility actions via the platform will generate revenues.
Suppliers that order flexibility action via the platform will generate both costs and revenues, where the revenues come from interacting with external markets.
Costs and revenues are calculated based on an internal method for profit sharing (TODO: explain).
Copy link
Contributor

@Flix6x Flix6x Mar 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This assumes a contractual relationship between multiple stakeholders, for example:

  • between Aggregator and Prosumer, where the Aggregator sells the balancing power to a third party and shares the profits with the Prosumer according to some contracted method for profit sharing. In this case the stated costs and revenues for the Prosumer may be after deducting the Aggregator fee (which typically include price components per flex activation and price components per unit of time, but may include arbitrarily complex price components).
  • between ESCo and Prosumer, where the ESCo advises the Prosumer to optimise against e.g. dynamic prices. Likewise, stated numbers may be after deducting the ESCo fee.

Alternatively, we can assume that all profit from activating flexibility goes to the Prosumer, or simply report the profits before sharing (and before deducting any service fees).

Copy link
Contributor Author

@nhoening nhoening Mar 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will use this content in my work on #45, thanks.

documentation/views/portfolio.rst Outdated Show resolved Hide resolved
@nhoening nhoening removed a link to an issue Mar 19, 2021
@nhoening nhoening merged commit f58a407 into main Mar 19, 2021
2 checks passed
@nhoening nhoening deleted the issue-46-Show_screenshots_in_documentation branch Mar 19, 2021
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

Successfully merging this pull request may close these issues.

None yet

2 participants