Skip to content
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: genetec camera integration #225

Merged
merged 5 commits into from
Jan 13, 2025
Merged

feat: genetec camera integration #225

merged 5 commits into from
Jan 13, 2025

Conversation

danleyb2
Copy link
Collaborator

@danleyb2 danleyb2 commented Jan 7, 2025

Summary by CodeRabbit

  • New Feature: Introduced Docker support for the Genetec camera integration, enabling easy deployment and setup.
  • Documentation: Added comprehensive setup instructions for integrating Genetec cameras with ParkPow and Snapshot services.
  • New Feature: Implemented Snapshot and ParkPowApi classes to facilitate interaction with external APIs for image recognition and vehicle data logging.
  • New Feature: Developed a Python script to handle Genetec webhooks and process events using threading for concurrent task management.

Copy link

github-actions bot commented Jan 7, 2025

Image description CodeRabbit

Walkthrough

This update introduces a Dockerized Python application for Genetec camera integration with ParkPow and Snapshot services. It includes a Dockerfile, setup instructions, API interaction classes, and a main script to handle webhooks and process data concurrently.

Changes

Files Summary
parkpow/genetec/Dockerfile Introduces a Dockerfile for building a Python 3.11-based container with necessary dependencies.
parkpow/genetec/README.md Provides setup instructions for Genetec camera integration and Docker image usage.
parkpow/genetec/api.py Adds Snapshot and ParkPowApi classes for interacting with external APIs using HTTP requests.
parkpow/genetec/main.py Implements a script to receive and process Genetec webhooks, forwarding data to external services.

In the realm of code where cameras see,
A Docker image sails on Python's sea.
With APIs that talk and scripts that run,
Integration's dance has just begun.
Webhooks whisper in threads so light,
ParkPow and Snapshot take flight! 🚀🐇


Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Commits Files that changed from the base of the PR and between 0a8b126 and f8c609b commits.
Files selected (4)
  • parkpow/genetec/Dockerfile (1)
  • parkpow/genetec/README.md (1)
  • parkpow/genetec/api.py (1)
  • parkpow/genetec/main.py (1)
Files skipped from review due to trivial changes (2)
  • parkpow/genetec/README.md
  • parkpow/genetec/main.py (diff too large)
Review comments generated (54)
  • Review: 3
  • LGTM: 51

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

parkpow/genetec/api.py Outdated Show resolved Hide resolved
parkpow/genetec/api.py Outdated Show resolved Hide resolved
parkpow/genetec/api.py Outdated Show resolved Hide resolved
@marcbelmont
Copy link
Collaborator

@dibonjohnseron
Can you please review this PR?

Copy link
Contributor

@dibonjohnseron dibonjohnseron left a comment

Choose a reason for hiding this comment

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

Looks good to me.

parkpow/genetec/main.py Outdated Show resolved Hide resolved
parkpow/genetec/main.py Show resolved Hide resolved
@marcbelmont
Copy link
Collaborator

@danleyb2
Can we merge this?

@danleyb2
Copy link
Collaborator Author

@marcbelmont yes

@marcbelmont marcbelmont merged commit f636f5a into master Jan 13, 2025
@marcbelmont marcbelmont deleted the genetec-pp-snapshot branch January 13, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants