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 ApexCharts configuration to README #16

Open
rany2 opened this issue Jun 9, 2024 · 4 comments
Open

Add ApexCharts configuration to README #16

rany2 opened this issue Jun 9, 2024 · 4 comments

Comments

@rany2
Copy link
Owner

rany2 commented Jun 9, 2024

Credit to @elRadix, I have permission to add it to the README
See https://community.home-assistant.io/t/open-meteo-solar-forecast/733073/133?u=rany

@rany2
Copy link
Owner Author

rany2 commented Jun 9, 2024

Make sure to add this suggestion by @andreas-bulling: https://community.home-assistant.io/t/open-meteo-solar-forecast/733073/136?u=rany

Without it the graph is more difficult to read (it shows a dotted red vertical line that tells you were you are in the graph right now).

@rany2
Copy link
Owner Author

rany2 commented Jun 9, 2024

Also I'll use watts attribute not wh_period, I'm not sure why everyone settled on the latter....

@andreas-bulling
Copy link

I can only speak for myself but the reason is that these are all estimates after all, i.e. they have error. What's the point in showing more fine-grained estimates (and errors) in this case?

@rany2
Copy link
Owner Author

rany2 commented Jun 10, 2024

For me it's mostly aesthetics, there's obviously no real advantage besides a smoother looking graph. wh_period looks more rough. If there is a technical reason to use wh_period instead, I'd be interested to hear it.

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

No branches or pull requests

2 participants