Added ONVIF camera component by matt2005 · Pull Request #7979 · home-assistant/core

matt2005 · 2017-06-09T20:38:21Z

Description:

Added support for ONVIF IP camera's. This discovers the stream url via ONVIF.

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

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

camera:
    platform: onvif
    host: 192.168.1.111
    name: onviftest

Checklist:

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:

Local tests with tox run successfully. Your PR cannot be merged unless tests pass
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.

homeassistant · 2017-06-09T21:08:29Z

Hi @matt2005,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

houndci-bot · 2017-06-09T21:08:33Z

too many blank lines (2)

houndci-bot · 2017-06-09T21:08:33Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:08:33Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:08:34Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:08:34Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:08:34Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:08:34Z

local variable 'media_profile' is assigned to but never used

houndci-bot · 2017-06-09T21:08:34Z

missing whitespace after ','
line too long (92 > 79 characters)

houndci-bot · 2017-06-09T21:08:34Z

missing whitespace after ','
line too long (192 > 79 characters)

houndci-bot · 2017-06-09T21:08:34Z

line too long (84 > 79 characters)

houndci-bot · 2017-06-09T21:16:05Z

continuation line over-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line over-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line over-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line over-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line over-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:16:05Z

continuation line under-indented for visual indent
trailing whitespace

houndci-bot · 2017-06-09T21:16:05Z

trailing whitespace

houndci-bot · 2017-06-09T21:16:06Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:22:26Z

continuation line under-indented for visual indent

houndci-bot · 2017-06-09T21:22:26Z

trailing whitespace

fabaff · 2017-06-10T20:49:43Z

You need to rebase your branch and clean-up the requirements.

matt2005 · 2017-06-10T22:49:40Z

rebased and rerun script/gen_requirements_all.py

matt2005 · 2017-06-12T21:02:33Z

rebased again and generated requirements. hopefully it's good to go now.

pvizeli

Looks okay. A small comments

pvizeli · 2017-06-13T15:37:36Z

Please cleanup the debugs, it is ok for test but to mutch for the end product.

I've removed the debug logs.

houndci-bot · 2017-06-13T19:00:13Z

continuation line under-indented for visual indent

matt2005 · 2017-06-13T19:00:18Z

Removed Debug logging

matt2005 · 2017-06-13T19:45:50Z

Requested changes done

Br3nda · 2017-06-13T22:00:22Z

Most components are using "host" not "input" in the config yml. Is this a relic from it being a ffmpeg input config before? Would host be better?

pvizeli

Need some small cleanups

pvizeli · 2017-06-13T22:33:56Z

Remove this

pvizeli · 2017-06-13T22:34:18Z

Remove this

pvizeli · 2017-06-13T22:35:51Z

pvizeli · 2017-06-13T22:36:21Z

pvizeli · 2017-06-13T22:36:41Z

Is not need to store

pvizeli · 2017-06-13T22:36:47Z

Is not need to store

pvizeli · 2017-06-13T22:37:23Z

Set this to debug level

houndci-bot · 2017-06-14T21:38:24Z

line too long (85 > 79 characters)

houndci-bot · 2017-06-14T21:38:24Z

line too long (84 > 79 characters)

houndci-bot · 2017-06-14T21:38:24Z

trailing whitespace

houndci-bot · 2017-06-14T21:38:24Z

undefined name 'CONF_FFMPEG_ARGUMENTS'

houndci-bot · 2017-06-14T21:38:24Z

undefined name 'CONF_FFMPEG_ARGUMENTS'
line too long (85 > 79 characters)

houndci-bot · 2017-06-14T21:38:24Z

missing whitespace around operator

pvizeli · 2017-06-15T09:08:17Z

Use the Format pattern for all params, don't mix + and format

andrey-git · 2018-07-02T07:34:10Z

I know I'm a year late to the party, but we shouldn't have added a new non-pip dependency.

I'm going to create homeassistant-specific package for this :(

matt2005 mentioned this pull request Jun 9, 2017

added ONVIF camera home-assistant/home-assistant.io#2794

Merged

homeassistant added the cla-needed label Jun 9, 2017

balloobbot added platform requirement labels Jun 9, 2017

houndci-bot reviewed Jun 9, 2017

View reviewed changes

homeassistant added cla-signed and removed cla-needed labels Jun 9, 2017

houndci-bot reviewed Jun 9, 2017

View reviewed changes

balloobbot added the component label Jun 10, 2017

homeassistant added cla-signed labels Jun 10, 2017

pvizeli requested changes Jun 13, 2017

View reviewed changes

homeassistant added the cla-signed label Jun 13, 2017

houndci-bot reviewed Jun 13, 2017

View reviewed changes

Comment thread homeassistant/components/camera/onvif.py Outdated

Copy link
Copy Markdown

houndci-bot Jun 13, 2017

Choose a reason for hiding this comment

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

continuation line under-indented for visual indent

balloobbot removed the component label Jun 13, 2017

pvizeli requested changes Jun 13, 2017

View reviewed changes

homeassistant added the cla-signed label Jun 14, 2017

houndci-bot reviewed Jun 14, 2017

View reviewed changes

homeassistant added the cla-signed label Jun 14, 2017

pvizeli requested changes Jun 15, 2017

View reviewed changes

matt2005 added 2 commits June 15, 2017 22:42

Added ONVIF camera component

3479716

added requirements

df30ac2

matt2005 added 20 commits June 15, 2017 22:42

updated requirements

e77512d

Added ONVIF camera component

f8d2bf3

added requirements

e54d734

corrected long lines

9f3d464

fixed indenting

f9f2695

fixed indenting

bad633a

removed bad whitespace

d50b6a9

updated requirements

215a5a8

fixed pylink error indenting

876dac4

rebased and fixed requirements

3d8e2d3

Removed Debug logging

96bf6ec

Added info logging to show URL being used.

d035bf2

corrected spacing

2b0c0d1

Tidied up and renamed input to host

568cf3e

fixed typo

abb89c5

corrected line lengths

0087206

added default to ffmpeg_arguments

a41970b

removed unecessary ffmpeg arguements

6f5b659

changed to use .format instead of +

3767992

fixed indenting

689a0a9

homeassistant added the cla-signed label Jun 15, 2017

cleanup & make it more readable

457c525

pvizeli approved these changes Jun 15, 2017

View reviewed changes

pvizeli added this to the 0.47 milestone Jun 15, 2017

balloob merged commit 0eaad46 into home-assistant:dev Jun 16, 2017

balloob modified the milestone: 0.47 Jun 16, 2017

balloob mentioned this pull request Jun 16, 2017

0.47 #8055

Merged

home-assistant locked and limited conversation to collaborators Oct 20, 2017

Uh oh!

Conversation

matt2005 commented Jun 9, 2017 • 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!

homeassistant commented Jun 9, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

fabaff commented Jun 10, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

matt2005 commented Jun 10, 2017

Uh oh!

matt2005 commented Jun 12, 2017

Uh oh!

pvizeli left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

matt2005 commented Jun 13, 2017

Uh oh!

matt2005 commented Jun 13, 2017

Uh oh!

Br3nda commented Jun 13, 2017

Uh oh!

pvizeli left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

matt2005 commented Jun 9, 2017 •

edited

Loading

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

fabaff commented Jun 10, 2017 •

edited

Loading