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

Error in docs #734

Open
simnh opened this issue Dec 14, 2020 · 5 comments
Open

Error in docs #734

simnh opened this issue Dec 14, 2020 · 5 comments
Assignees

Comments

@simnh
Copy link
Member

simnh commented Dec 14, 2020

Hey I found these lines:

Connect the invest variables of the input and the output flow.
.. math::
InvestmentFlow.invest(source(n), n) + existing = \\
(InvestmentFlow.invest(n, target(n)) + existing) * \\
invest\_relation\_input\_output(n) \\
\forall n \in \textrm{INVEST\_REL\_IN\_OUT}

Should they be there, as it is not the Storage Investment Block? So there is no investment. Maybe copy paste error?

@uvchik
Copy link
Member

uvchik commented Dec 14, 2020

Thanks @simnh Could you open a PR?

@simnh
Copy link
Member Author

simnh commented Dec 14, 2020

There is another error I found:

.. math::
E(t) \leq E_{invest} \cdot c_{max}(t)
* :attr:`min_storage_level`
Rule for lower bound constraint for the storage content:
.. math:: E(t) \geq E_{invest} \cdot c_{min}(t)

I can open a PR ...

@joroeder
Copy link
Member

There is another error I found:

.. math::
E(t) \leq E_{invest} \cdot c_{max}(t)
* :attr:`min_storage_level`
Rule for lower bound constraint for the storage content:
.. math:: E(t) \geq E_{invest} \cdot c_{min}(t)

I can open a PR ...

What is the error? The existing capacity is missing, it must be: (E_{invest} + E_{exist}) ?

@simnh
Copy link
Member Author

simnh commented Dec 14, 2020

Yes I think so, I opened a PR.

@joroeder
Copy link
Member

Hey I found these lines:

Connect the invest variables of the input and the output flow.
.. math::
InvestmentFlow.invest(source(n), n) + existing = \\
(InvestmentFlow.invest(n, target(n)) + existing) * \\
invest\_relation\_input\_output(n) \\
\forall n \in \textrm{INVEST\_REL\_IN\_OUT}

Should they be there, as it is not the Storage Investment Block? So there is no investment. Maybe copy paste error?

Actually, there is such a constraint in the GenericStorageBlock:

def _power_coupled(block, n):

Maybe, it is supposed for some cases if you have a fixed nominal storage capacity and an investment Flow at In-and Outflow? couple these?

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

No branches or pull requests

4 participants