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

Add info about left calculation method #294

Closed
wants to merge 4 commits into from
Closed

Add info about left calculation method #294

wants to merge 4 commits into from

Conversation

ahmadyusuf
Copy link
Contributor

Added a comment to use the left calculation method if the house has solar panels installed as the default trapezoidal calculates wrong values if the consumpiton is 0 for an extended period of time

Added a comment to use the left calculation method if the house has solar panels installed as the default trapezoidal calculates wrong values if the consumpiton is 0 for an extended period of time
@ahmadyusuf
Copy link
Contributor Author

Please refer to
#274

@ahmadyusuf ahmadyusuf marked this pull request as draft June 11, 2023 13:49
@ahmadyusuf ahmadyusuf closed this Jun 11, 2023
@ahmadyusuf ahmadyusuf reopened this Jun 11, 2023
@ahmadyusuf ahmadyusuf marked this pull request as ready for review June 11, 2023 13:55
@klaasnicolaas
Copy link
Owner

I would prefer to omit the comment piece and include it in the FAQ

@klaasnicolaas klaasnicolaas added the enhancement Enhancement of the code, not introducing new features. label Jun 19, 2023
@ahmadyusuf
Copy link
Contributor Author

Sure, will do that
Thank you

@ahmadyusuf ahmadyusuf closed this Jun 20, 2023
@klaasnicolaas
Copy link
Owner

Closing the PR wouldn't really have been necessary, you can just make and push a commit 😅

@ahmadyusuf ahmadyusuf reopened this Jun 21, 2023
@ahmadyusuf
Copy link
Contributor Author

Done, sorry I am new to this

@klaasnicolaas klaasnicolaas changed the title Update home_assistant_glow.yaml Add info about left calculation method Jun 21, 2023
@github-actions
Copy link

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale There has not been activity on this issue or PR for quite some time. label Jul 22, 2023
@klaasnicolaas klaasnicolaas added no-stale This issue or PR is exempted from the stable bot. and removed stale There has not been activity on this issue or PR for quite some time. labels Jul 22, 2023
@klaasnicolaas klaasnicolaas self-assigned this Jul 22, 2023
@klaasnicolaas
Copy link
Owner

Could you perhaps convert this FAQ to the new format on the documentation website, these will no longer go into the readme file.

@ahmadyusuf ahmadyusuf closed this by deleting the head repository Feb 11, 2024
@ahmadyusuf
Copy link
Contributor Author

Sure, I will close this pull request as the branch was out of sync and created a new one

#392

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of the code, not introducing new features. no-stale This issue or PR is exempted from the stable bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants