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

Fix green share calculation #11545

Merged
merged 10 commits into from
Jan 9, 2024
Merged

Fix green share calculation #11545

merged 10 commits into from
Jan 9, 2024

Conversation

MarkusGH
Copy link
Contributor

@MarkusGH MarkusGH commented Jan 7, 2024

Fix #11032

Fix green share calculation to ensure loadpoints are "last in line" regarding solar power consumption

Fix green share calculation
Fox calculation formula
Fix formatting
@andig andig requested a review from naltatis January 7, 2024 14:59
@andig andig added the enhancement New feature or request label Jan 7, 2024
@andig
Copy link
Member

andig commented Jan 7, 2024

@MarkusGH stell Deinen PRs bitte im Text mal ein

Fix #...

voran, dann werden sie automatisch mit den Issues verlinkt und diese bei Merge auch geschlossen.

@naltatis
Copy link
Member

naltatis commented Jan 8, 2024

Super. Magst du noch nen Testcase mit dem im Issue beschriebenen Szenario ergänzen?

@MarkusGH
Copy link
Contributor Author

MarkusGH commented Jan 8, 2024

Super. Magst du noch nen Testcase mit dem im Issue beschriebenen Szenario ergänzen?

Wollen kann man nicht gerade sagen :-)
War aber besser so, sowohl in den Tests als auch in der Berechnung waren noch Fehler.

Copy link
Member

@naltatis naltatis left a comment

Choose a reason for hiding this comment

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

💚 Looks good.

@MarkusGH ich hab noch mal die Zahlen im Test vereinfacht und dem test nen inhaltichen Namen gegeben. Den Connect zum Issue bekommen wir über die Git Historie automatisch durch diesen PR. Also falls man sich später mach fragt was genau der Anwendungsfall war.
Wenn du keine Einwände hast würd ich das reinnehmen.

@naltatis naltatis merged commit 2471d94 into evcc-io:master Jan 9, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Potentially incorrect Charging Cost Calculation
3 participants