Skip to content

Fix transition events with zero accuracy.#2832

Closed
pavoni wants to merge 1 commit into
devfrom
fix_owntracks_beacon_accuracy_bug2
Closed

Fix transition events with zero accuracy.#2832
pavoni wants to merge 1 commit into
devfrom
fix_owntracks_beacon_accuracy_bug2

Conversation

@pavoni
Copy link
Copy Markdown
Contributor

@pavoni pavoni commented Aug 15, 2016

Description:
Zero accuracy checks prevented beacon transitions working properly

Related issue (if applicable): fixes #2830

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#

Example entry for configuration.yaml (if applicable):

Checklist:

If user exposed functionality or configuration variables are added/changed:

If code communicates with devices, web services, or a:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

If the code does not interact with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • Tests have been added to verify that the new code works.

# But does exit region correctly
self.assertFalse(owntracks.REGIONS_ENTERED[USER])

def test_event_entry_exit_zero_accuracy(self):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

farcy v1.1

  • D102: Missing docstring in public method

@pavoni
Copy link
Copy Markdown
Contributor Author

pavoni commented Aug 15, 2016

Having a bad day - picked up the gui commit!

@pavoni pavoni closed this Aug 15, 2016
@pavoni pavoni deleted the fix_owntracks_beacon_accuracy_bug2 branch August 15, 2016 12:12
@home-assistant home-assistant locked and limited conversation to collaborators Mar 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Owntracks enter/leave beacon transitions not working

2 participants