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

Multidevice support & read frequency #14

Closed
1 task done
Marquo1 opened this issue Jun 10, 2022 · 7 comments
Closed
1 task done

Multidevice support & read frequency #14

Marquo1 opened this issue Jun 10, 2022 · 7 comments
Labels
enhancement New feature or request

Comments

@Marquo1
Copy link

Marquo1 commented Jun 10, 2022

Did you read the instructions?

The request

Firstly, I would also like to report a new device working with the integration. In my case it is Kamstrup Multical 601. Thank you for this integration!

Secondly, can you please specify what's the read frequency and ideally make it a configurable value?

Additional information

From my research, I understand that reading the meter drains its battery (~15 min of lifetime reduction per read with brand new battery lasting 10 years without readouts). It'd be good to have control over it if ones device is battery powered.

@Marquo1 Marquo1 added the enhancement New feature or request label Jun 10, 2022
@golles
Copy link
Owner

golles commented Jun 10, 2022

Hi @Marquo1

Thanks for reporting that this component works on the 601, I'll update the docs

The current update interval is 60 seconds:
https://github.com/golles/ha-kamstrup_403/blob/main/custom_components/kamstrup_403/__init__.py#L29

I can imagine that this is not suitable for battery-operated devices.
I'll make this configurable in the future

@golles
Copy link
Owner

golles commented Jun 11, 2022

Hi @Marquo1 I've made the change you suggested in #15
Would it be possible to help test this change? You can do this by checking out the configure_scan_interval branch.

I'm testing this now as well, if all is fine I'll release this later this weekend.

@Marquo1
Copy link
Author

Marquo1 commented Jun 11, 2022

@golles Splendid!

I'll try it in the morning and come back to you

@Marquo1
Copy link
Author

Marquo1 commented Jun 12, 2022

Hi @golles

Everything is working but it was hard to confirm if the scan interval was changed. It seems the entities only show they're updated when the value has changed. Timing the updates with a stopwatch while my SO is showering doesn't feel creepy at all 🤣

@golles
Copy link
Owner

golles commented Jun 12, 2022

I noticed an issue that I've fixed 2 hours ago.

I've tested it by changing the value to 10 seconds and I could see updates every 10 seconds. I had my logger set as:

logger:
  default: warn
  logs:
    custom_components.kamstrup_403: debug

@golles
Copy link
Owner

golles commented Jun 12, 2022

I've merged the PR and released 1.2.0 https://github.com/golles/ha-kamstrup_403/releases/tag/1.2.0

Thank you for the suggestion for this option and confirming that it works on the 601 meter.

Please star this repository if you're happy with this component - it makes me happy :-)

@golles golles closed this as completed Jun 12, 2022
@Marquo1
Copy link
Author

Marquo1 commented Jun 12, 2022

I've tested it by changing the value to 10 seconds and I could see updates every 10 seconds. I had my logger set as:

logger:
  default: warn
  logs:
    custom_components.kamstrup_403: debug

Ah that's experience I don't have. ;) Thanks for listening!

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

No branches or pull requests

2 participants