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

Conversion constraint documentation #470

Open
SisiLimperatrice opened this issue May 2, 2023 · 1 comment
Open

Conversion constraint documentation #470

SisiLimperatrice opened this issue May 2, 2023 · 1 comment
Labels

Comments

@SisiLimperatrice
Copy link

SisiLimperatrice commented May 2, 2023

Problem description

I spotted a small error in the mathematical formulation documentation:
Under the Conversion section of the mathematical formulation, when dexcribing balance_conversion_constraint_rule, the efficiency is applied two times both for the carrier production and consumption. The code seems to be right but you should remove the multiplication by the efficiency for the carrier production.

Calliope version

Stable documentation version

@brynpickering
Copy link
Member

Thanks @SisiLimperatrice. This should be fixed in our new constraint documentation (see here). Hopefully it'll be the stable documentation in the next months.

You could still update version 0.6 documentation by updating this docstring in a Pull Request to the 0.6 branch.

@brynpickering brynpickering added the v0.6 Version 0.6 label Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants