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

Updated behavior on lost connections. #49

Closed
wants to merge 80 commits into from
Closed

Updated behavior on lost connections. #49

wants to merge 80 commits into from

Conversation

pawlizio
Copy link
Collaborator

@pawlizio pawlizio commented Aug 4, 2020

  • Closing of transport after a connection has been lost.
  • Stopping heartbeat if a connection has been lost.
  • Starting heartbeat after successful reconnection.
  • Reduction of Logger warnings.
  • Increasing of debug messages.

Unfortunately it will nox fix #30.

@coveralls
Copy link

coveralls commented Aug 4, 2020

Coverage Status

Coverage decreased (-0.06%) to 81.56% when pulling 7c26df1 on pawlizio:master into f579c06 on Julius2342:master.

@pawlizio pawlizio marked this pull request as ready for review August 10, 2020 09:01
@pawlizio pawlizio marked this pull request as draft August 10, 2020 09:11
@pawlizio pawlizio marked this pull request as ready for review August 10, 2020 09:14
@pawlizio
Copy link
Collaborator Author

@Julius2342: Did you ever try to discover the KLF200 via zeroconf? On the version 18 of the KLF200 API specification on chapter "15 Appendix 3: Identifying IP address of a KLF200 device using mDNS protocol." there is written a procedure. However trying this I figured out some strange behaviour which I could not allocate to either VELUX or python zeroconf integration. I have the following issue: python-zeroconf/python-zeroconf#288

@Julius2342
Copy link
Owner

No, i never tried zeroconf with my velux.

@pawlizio pawlizio marked this pull request as draft November 13, 2020 10:51
	pyvlx/frames/frame_facory_default.py
	pyvlx/frames/frame_get_local_time.py
	pyvlx/frames/frame_get_network_setup.py
	pyvlx/frames/frame_leave_learn_state.py
	pyvlx/frames/frame_password_change.py
Add Api Functionality for KLF200 Commands
@pawlizio pawlizio closed this Dec 20, 2020
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.

Keeping connection open freezes KLF
4 participants