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
Don't pass in loop #23984
Don't pass in loop #23984
Conversation
Better regexp: Pretty much all the ones that aren't the last argument are to external libraries. There's a few calls to the |
else: | ||
reader_factory = partial( | ||
create_dsmr_reader, config[CONF_PORT], config[CONF_DSMR_VERSION], | ||
update_entities_telegram, loop=hass.loop) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This breaks the dsmr module. Should this be replaced with loop=asyncio.get_event_loop()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll revert it, my bad.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reinstating this should be a temporary fix. The loop argument should always be optional (since it's going to go away anyway).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@smurfix: it's actually caused by an upstream bug, as soon as that's fixed we can fix this properly: ndokter/dsmr_parser#36
Description:
First pass removing passing loop around.
Related issue (if applicable): home-assistant/architecture#238
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for
configuration.yaml
(if applicable):Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
python3 -m script.hassfest
.requirements_all.txt
by runningpython3 -m script.gen_requirements_all
..coveragerc
.If the code does not interact with devices: