-
Notifications
You must be signed in to change notification settings - Fork 52
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
feat: support for usb cameras and onvif zoom functionality in camera mgmt app #184
feat: support for usb cameras and onvif zoom functionality in camera mgmt app #184
Conversation
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 have looked into the zoom implementation part, some of the files need the copyright update.
application-services/custom/camera-management/appcamera/commands.go
Outdated
Show resolved
Hide resolved
application-services/custom/camera-management/appcamera/evam.go
Outdated
Show resolved
Hide resolved
application-services/custom/camera-management/appcamera/evam.go
Outdated
Show resolved
Hide resolved
...ervices/custom/camera-management/web-ui/src/app/camera-selector/camera-selector.component.ts
Outdated
Show resolved
Hide resolved
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.
LGTM
a426086
a426086
to
cbc42d3
Compare
Squashed and fixed DCO |
application-services/custom/camera-management/appcamera/evam.go
Outdated
Show resolved
Hide resolved
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.
LGTM
Most of it looks good. I have added a comment for the docs. |
ac26eb3
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.
LGTM
…mgmt app Signed-off-by: Anthony Casagrande <[email protected]> Signed-off-by: Christian Darr <[email protected]> Signed-off-by: Brian Osburn <[email protected]> Co-authored-by: Brian Osburn <[email protected]> Co-authored-by: Christian Darr <[email protected]>
8c10a6e
to
460dac7
Compare
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.
LGTM
Signed-off-by: Bryon Nevis <[email protected]> Signed-off-by: Bryon Nevis <[email protected]> fix: remove duplicated path from query uri (edgexfoundry#182) Signed-off-by: Anthony Casagrande <[email protected]> fix: Modify pan tilt function by getting pan/tilt range dynamically (edgexfoundry#181) * fix: Get pan/tilt values dynamically Signed-off-by: preethi-satishcandra <[email protected]> feat: support for usb cameras and onvif zoom functionality in camera mgmt app (edgexfoundry#184) Signed-off-by: Anthony Casagrande <[email protected]> Signed-off-by: Christian Darr <[email protected]> Signed-off-by: Brian Osburn <[email protected]> Co-authored-by: Brian Osburn <[email protected]> Co-authored-by: Christian Darr <[email protected]> feat: updated app service with target type Signed-off-by: brianointel <[email protected]> feat: added code for pipeline function Signed-off-by: brianointel <[email protected]> feat: template code for funtion pipelines by topic Signed-off-by: brianointel <[email protected]> feat: added initial event handler Signed-off-by: brianointel <[email protected]> feat: more device messagebus progress Signed-off-by: brianointel <[email protected]> feat: message bus code Signed-off-by: Brian Osburn <[email protected]> feat: messagebus progress Signed-off-by: Brian Osburn <[email protected]> feat: created defaultpipelinefunction Signed-off-by: Brian Osburn <[email protected]> feat: try to start pipelines on startup Signed-off-by: brianointel <[email protected]> feat: small changes Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: added config params Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: handle device delete event to stop pipeline also some minor refactoring Signed-off-by: Anthony Casagrande <[email protected]>
* feat: Update device service example for Levski Signed-off-by: Leonard Goodell <[email protected]> build: Update HELM Chart to Levski released images (edgexfoundry#140) Signed-off-by: Leonard Goodell <[email protected]> Signed-off-by: Leonard Goodell <[email protected]> feat: added support for shared and non-shared volumes Close edgexfoundry#92 Signed-off-by: Kyle Morton <[email protected]> fix: EdgeX UI in kubernetes was not binding container network IF Signed-off-by: Bryon Nevis <[email protected]> feat: Added support for onvif and usb camera Closes edgexfoundry#107 Signed-off-by: Kyle Morton <[email protected]> feat: Added support for modbus-simulator Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for ds-mqtt and mqtt-broker Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for ds-modbus Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for ds-bacnet Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for ds-snmp Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for ds-rfid-llrp Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for ds-gpio Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for asc-mqtt and asc-httpexport Signed-off-by: Kyle Morton <[email protected]> feat: Add support in helm for asc-sample, asc-metrics, and as-llrp Signed-off-by: Kyle Morton <[email protected]> fix: Updated device-onvif-camera and device-usb-camera directory and volume (edgexfoundry#156) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> fix: added service for edgex-ui (edgexfoundry#162) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> feat: Added support for nats (edgexfoundry#163) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> fix: Updated health checks to appropriate protocol (edgexfoundry#164) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> fix: added kuiper-connections and kuipers-sources volumes (edgexfoundry#165) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> fix: Correct addConsulRoles list (edgexfoundry#166) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> feat: Support optional hostPort binding (edgexfoundry#167) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> feat: Expose services on a per-service basis (edgexfoundry#169) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> fix: Added container port names (edgexfoundry#170) Signed-off-by: Kyle Morton <[email protected]> Signed-off-by: Kyle Morton <[email protected]> Co-authored-by: Kyle Morton <[email protected]> feat!: Remove system-mgmt-agent (deprecated) (edgexfoundry#175) BREAKING CHANGE: Remove system-mgmt-agent (deprectated) Signed-off-by: Bryon Nevis <[email protected]> Signed-off-by: Bryon Nevis <[email protected]> docs: add link to camera app example video Signed-off-by: Anthony Casagrande <[email protected]> feat: Add Helm security scanners (edgexfoundry#174) Signed-off-by: Bryon Nevis <[email protected]> Signed-off-by: Bryon Nevis <[email protected]> fix: remove duplicated path from query uri (edgexfoundry#182) Signed-off-by: Anthony Casagrande <[email protected]> fix: Modify pan tilt function by getting pan/tilt range dynamically (edgexfoundry#181) * fix: Get pan/tilt values dynamically Signed-off-by: preethi-satishcandra <[email protected]> feat: support for usb cameras and onvif zoom functionality in camera mgmt app (edgexfoundry#184) Signed-off-by: Anthony Casagrande <[email protected]> Signed-off-by: Christian Darr <[email protected]> Signed-off-by: Brian Osburn <[email protected]> Co-authored-by: Brian Osburn <[email protected]> Co-authored-by: Christian Darr <[email protected]> feat: updated app service with target type Signed-off-by: brianointel <[email protected]> feat: added code for pipeline function Signed-off-by: brianointel <[email protected]> feat: template code for funtion pipelines by topic Signed-off-by: brianointel <[email protected]> feat: added initial event handler Signed-off-by: brianointel <[email protected]> feat: more device messagebus progress Signed-off-by: brianointel <[email protected]> feat: message bus code Signed-off-by: Brian Osburn <[email protected]> feat: messagebus progress Signed-off-by: Brian Osburn <[email protected]> feat: created defaultpipelinefunction Signed-off-by: Brian Osburn <[email protected]> feat: try to start pipelines on startup Signed-off-by: brianointel <[email protected]> feat: small changes Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: added config params Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: handle device delete event to stop pipeline Signed-off-by: brianointel <[email protected]> also some minor refactoring Signed-off-by: Anthony Casagrande <[email protected]> fix: pr updates Signed-off-by: Brian Osburn <[email protected]> refactor: fix imports from merge Signed-off-by: Anthony Casagrande <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> fix: updated funtion name Signed-off-by: Brian Osburn <[email protected]> feat: Add Helm security scanners (edgexfoundry#174) Signed-off-by: Bryon Nevis <[email protected]> Signed-off-by: Bryon Nevis <[email protected]> fix: remove duplicated path from query uri (edgexfoundry#182) Signed-off-by: Anthony Casagrande <[email protected]> fix: Modify pan tilt function by getting pan/tilt range dynamically (edgexfoundry#181) * fix: Get pan/tilt values dynamically Signed-off-by: preethi-satishcandra <[email protected]> feat: support for usb cameras and onvif zoom functionality in camera mgmt app (edgexfoundry#184) Signed-off-by: Anthony Casagrande <[email protected]> Signed-off-by: Christian Darr <[email protected]> Signed-off-by: Brian Osburn <[email protected]> Co-authored-by: Brian Osburn <[email protected]> Co-authored-by: Christian Darr <[email protected]> feat: updated app service with target type Signed-off-by: brianointel <[email protected]> feat: added code for pipeline function Signed-off-by: brianointel <[email protected]> feat: template code for funtion pipelines by topic Signed-off-by: brianointel <[email protected]> feat: added initial event handler Signed-off-by: brianointel <[email protected]> feat: more device messagebus progress Signed-off-by: brianointel <[email protected]> feat: message bus code Signed-off-by: Brian Osburn <[email protected]> feat: messagebus progress Signed-off-by: Brian Osburn <[email protected]> feat: created defaultpipelinefunction Signed-off-by: Brian Osburn <[email protected]> feat: try to start pipelines on startup Signed-off-by: Brian Osburn <[email protected]> feat: small changes Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: added config params Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> feat: handle device delete event to stop pipeline also some minor refactoring Signed-off-by: Anthony Casagrande <[email protected]> fix: pr updates Signed-off-by: Brian Osburn <[email protected]> refactor: fix imports from merge Signed-off-by: Anthony Casagrande <[email protected]> feat: pr updates Signed-off-by: Brian Osburn <[email protected]> fix: updated funtion name Signed-off-by: Brian Osburn <[email protected]> fix: corrected error message placement Signed-off-by: brianointel <[email protected]>
If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:
Testing Instructions
NOTE: Not fixing DCO/squashing/force-pushing until reviews are completed. This helps the people who have already reviewed it to better track what has changed.