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

Attempt to fix #20

Merged
merged 3 commits into from
Feb 10, 2022
Merged

Attempt to fix #20

merged 3 commits into from
Feb 10, 2022

Conversation

tgcowell
Copy link
Contributor

Hi ....

Please review as I'm new to this and just giving it a crack, in an ideal world all the attributes would become sensors... and I needed to provide auth for my MQTT host and your options don't provide this.

Thought I would give it a shot?

Let me know your thoughts.

Added Daily Purchased Energy & Daily energy returned to grid.
Updated to include the MQTT options
@sEdsKi
Copy link

sEdsKi commented Feb 1, 2022

Yes my log is showing all of this information - but no sensors to use it?

INFO:root:{'inverter_size': 8.0, 'daily_power_yield': 53500.0, 'total_power_yield': 86.52, 'inverter_power_on_hours': 5064, 'internal_temp': 51.2, 'pv1_voltage': 168.7, 'pv1_current': 0.6, 'pv2_voltage': 305.0, 'pv2_current': 0.6, 'total_pv_power': 308, 'grid_voltage': 244.2, 'inverter_current': 1.2, 'total_active_power': 300, 'grid_frequency': 50.0, 'export_power': 281, 'export_power_indicator': 0, 'power_meter': 589, 'daily_export_energy': 29600.0, 'total_export_energy': 5920.7, 'daily_import_energy': 7400.0, 'total_import_energy': 4531.3, 'daily_energy_consumption': 23900.0, 'total_energy_consumption': 2731.6, 'timestamp': '2022-02-01T18:05:23'}
INFO:root:Published to MQTT

@tgcowell
Copy link
Contributor Author

tgcowell commented Feb 1, 2022

They appear as attributes against any of the sensors, so you can manually create a sensor calling an attribute. This is how I've been able to get around it until the sensors are updated.

@sEdsKi
Copy link

sEdsKi commented Feb 1, 2022

@tgcowell Yes this is where Im at now. If I can piece together enough online help to write those sensors ill be pretty happy.

@ttvt ttvt merged commit 99b7fd2 into MatterVN:main Feb 10, 2022
@ttvt
Copy link
Contributor

ttvt commented Feb 10, 2022

The connection to home assistant's mqtt don't need the user name & password. If you need connect to different mqtt please use the solariot.

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

Successfully merging this pull request may close these issues.

3 participants