WIP Adds facebox_face_detect by robmarkcole · Pull Request #14279 · home-assistant/core

robmarkcole · 2018-05-04T06:32:24Z

Replaces #14248

Description:

Adds component for face detection (number of faces) using facebox. Run facebox with:

MB_KEY="INSERT-YOUR-KEY-HERE"

docker run -p 8080:8080 -e "MB_KEY=$MB_KEY" -e "MB_FACEBOX_DISABLE_RECOGNITION=true" machinebox/facebox

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

Example entry for `configuration.yaml` (if applicable):

image_processing:
  - platform: facebox_face_detect
    ip_address: 192.168.0.1
    port: 8080
    source:
      - entity_id: camera.local_file

Checklist:

The code change is tested and works locally.
Local tests pass with tox. Your PR cannot be merged unless tests pass

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

Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
New dependencies are only imported inside functions that use them ([example][ex-import]).
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:

Tests have been added to verify that the new code works.

* Fixed bug - unable to set base readaonly property * PR fixes * Added line

) * Fix so it is possible to ignore discovered config entry handlers * Improve efficiency

* Fix race condition for component loaded before listening * async/await syntax

* Broadlink switch, fixes issue #13799 * slugify

* Bumped pypollencom to 1.1.2 * Updated requirements_all.txt

This reverts commit 6fa60c4.

0.67.1

* Order the output of the automation editor * Lint

* Catch exception for KeyError * Use get and added test

* Handle HomeKit configuration failure more cleanly Add support for handling cases where HomeKit configuration fails, and give the user more information about what to do. * Don't consume the exception for a homekit.UnknownError If we get an UnknownError then we should alert the user but also still generate the backtrace so there's actually something for them to file in a bug report.

…14080) * fix timezone issue in tibber #14009 * remove debug print

* Setup all-linking service * Remove extra line * Remove linefeed and tab escape chars * Add services delete_all_link, load_all_link_database and print_all_link_database * Check if reload is set * Confirm entity is InsteonPLMEntity before attempting to load or print ALDB * Debug load and print ALDB * Debug print aldb * Debug print_aldb * Get entity via platform * Track Insteon entities in component * Store entity list in hass.data * Add entity to hass.data * Add ref to hass in InsteonPLMEntity * Pass hass correctly to InsteonPLMBinarySensor * Fix reference to ALDBStatus.PARTIAL * Print ALDB record as string * Get ALDB record from memory address * Reformat ALDB log output * Add print_im_aldb service * Remove reference to self in print_aldb_to_log * Remove reference to self in print_aldb_to_log * Fix spelling issue with load_all_link_database service * Bump insteonplm to 0.9.1 * Changes from code review * Code review changes * Fix syntax error * Correct reference to cv.boolean and update requirements * Update requirements * Fix flake8 errors * Reload as boolean test * Remove hass from entity init

# Conflicts: # homeassistant/components/image_processing/facebox_face_detect.py

robmarkcole · 2018-05-05T16:22:03Z

I've followed the dev instructions (to the best of my understanding)but I still repeatedly run into this issue of conflict and hundreds of other files being modified. Is there a bullet proof approach to avoid this?

MartinHjelmare · 2018-05-05T17:41:30Z

I often use the git rebase --onto command to make sure my branch is rebased on top of the tip of the target branch.

git rebase --onto dev LAST_COMMIT_BEFORE_YOUR_BRANCH YOUR_BRANCH

robmarkcole · 2018-05-05T17:50:37Z

Thanks for the advice @MartinHjelmare
Is there a way to repair this branch?

MartinHjelmare · 2018-05-05T18:09:28Z

Make sure you backup your changed code somehow first. Then you can try this:
https://stackoverflow.com/questions/134882/undoing-a-git-rebase

Disclaimer: I haven't tried it myself.

robmarkcole · 2018-05-06T04:49:20Z

This branch is a mess so I'm going to create a fresh one & PR

dlbroadfoot and others added 30 commits April 17, 2018 22:34

Fix Gogogate2 'available' attribute (#13728)

e836674

* Fixed bug - unable to set base readaonly property * PR fixes * Added line

Fix so it is possible to ignore discovered config entry handlers (#13741

0adb240

) * Fix so it is possible to ignore discovered config entry handlers * Improve efficiency

Prevent vesync doing I/O in event loop (#13862)

26c76e3

Fix #13846 Double underscore in bluetooth address (#13884)

727ab95

Fix race condition for component loaded before listening (#13887)

663aeb1

* Fix race condition for component loaded before listening * async/await syntax

Missing property decorator added (#13889)

bcd8a69

Fix call to parent broadlink switch (#13906)

6520635

* Broadlink switch, fixes issue #13799 * slugify

Import operation modes from air humidifier (#13908)

fadff18

Upgrade pyqwikswitch to 0.71 (#13920)

6fa60c4

Upgrade somecomfort to 0.5.2 (#13940)

5350682

Update pyhydroquebec to 2.2.2 (#13946)

e9b997d

Update pyfido to 2.1.1 (#13947)

6c456ad

Bumped pypollencom to 1.1.2 (#13959)

24ec8c5

* Bumped pypollencom to 1.1.2 * Updated requirements_all.txt

Version bump to 0.67.1

e7aea5c

Revert "Upgrade pyqwikswitch to 0.71 (#13920)"

c076dbe

This reverts commit 6fa60c4.

Merge pull request #13975 from home-assistant/rc

2cb9e2d

0.67.1

Merge branch 'dev' into rc

a566804

Version bump to 0.68.0b0

2b53729

Bump frontend to 20180425.0

8cb1e17

Qwikswitch binary sensors (#14008)

44be801

Order the output of the automation editor (#14019)

2bc87bf

* Order the output of the automation editor * Lint

HomeKit Alarm Control Panel Code Exception Fix (#14025)

cb839ef

* Catch exception for KeyError * Use get and added test

Revert cast platform polling mode (#14027)

fc1f6ee

Update device classes for contact sensor HomeKit (#14051)

2e3a27e

Version bump to 0.68.0b1

c497515

Bump frontend to 20180426

1b71ce3

Version bump to 0.68.0b2

833508f

Upgrade broadlink lib (#14074)

403a546

Fix timezone issue when calculating min/max values in tibber #14009 (#…

9d0251c

…14080) * fix timezone issue in tibber #14009 * remove debug print

robmarkcole requested review from a team, Kane610, cdce8p, dgomes, molobrakos, syssi and tinloaf as code owners May 5, 2018 15:04

homeassistant added the cla-signed label May 5, 2018

rsmeral and others added 11 commits May 5, 2018 17:06

Onkyo: SUPPORT_VOLUME_STEP (#14299)

2e8eaf4

Update translations

a4b6983

Update frontend to 20180505.0

83e342d

Merge branch 'master' into dev

2326312

Version bump to 0.70.0.dev0

f3411f8

Adds facebox face detect

e623e87

Remove ROUNDING_DECIMALS

eaf710f

Adds facebox face detect

5cbf7c9

Address rebase

f5cf660

Merge remote-tracking branch 'origin/facebox-detect' into facebox-detect

710f841

# Conflicts: # homeassistant/components/image_processing/facebox_face_detect.py

homeassistant added the cla-signed label May 5, 2018

robmarkcole closed this May 6, 2018

robmarkcole mentioned this pull request May 6, 2018

Adds facebox #14307

Closed

8 tasks

home-assistant locked and limited conversation to collaborators Sep 5, 2018

ghost added integration: facebox_face_detect and removed platform: image_processing.facebox_face_detect labels Mar 21, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

WIP Adds facebox_face_detect#14279

WIP Adds facebox_face_detect#14279
robmarkcole wants to merge 85 commits intohome-assistant:devfrom
robmarkcole:facebox-detect

robmarkcole commented May 4, 2018 •

edited

Loading

Uh oh!

robmarkcole commented May 5, 2018

Uh oh!

MartinHjelmare commented May 5, 2018

Uh oh!

robmarkcole commented May 5, 2018

Uh oh!

MartinHjelmare commented May 5, 2018

Uh oh!

robmarkcole commented May 6, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Uh oh!

Conversation

robmarkcole commented May 4, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description:

Example entry for configuration.yaml (if applicable):

Checklist:

Uh oh!

robmarkcole commented May 5, 2018

Uh oh!

MartinHjelmare commented May 5, 2018

Uh oh!

robmarkcole commented May 5, 2018

Uh oh!

MartinHjelmare commented May 5, 2018

Uh oh!

robmarkcole commented May 6, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

robmarkcole commented May 4, 2018 •

edited

Loading

Example entry for `configuration.yaml` (if applicable):