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

Adding a simple Ansible playbook and some updated notes/requirements, for easy deployment of monitor_run.py #3

Merged
merged 28 commits into from
Sep 28, 2024

Conversation

wombat70
Copy link
Contributor

I've written a basic Ansible playbook to automate (and document) some of the setup/configuration steps required to get the monitor_run.py script running on a Raspberry Pi with the provided yolov5 model. There is also a short README which outlines how to run the playbook.

For those who prefer to configure things manually, the playbook YAML file together with the revised requirements.txt file effectively documents the steps/tasks and required dependencies. This may also help resolve/workaround issue #1.

All the new files are in a new deploy directory and I've only made minor changes to the monitor.py script, namely:

  • Use filetype instead of imghdr, because the latter is being deprecated ref. PEP 594.
  • Added some OpenCV settings in an attempt to reduce capture latency when using a USB camera.
  • Use results.ims instead of results.imgs, because the yolov5 Detections class renamed its image-list property in Improved Profile() inference timing ultralytics/yolov5#9024.

Hope this helps! I'm looking forward to receiving any comments/suggestions or questions. :-)

Tom

…g yolov5 submodule to same commit as on vespai main branch.
@spienz
Copy link

spienz commented Sep 18, 2024

Hy,

works so far with Rasbian 64, but:
SSH Key Config should be getting a better description.

And with ansible the installation of opencv won't work. We had to install it manually with pip "opencv-python-headless"
Then it works fine so far.
We should implement a test picture function.
And also document the way to geat an IP Camerea working. We solved it for us.

Regards

Walter & Sebastian

Copy link
Owner

@andrw3000 andrw3000 left a comment

Choose a reason for hiding this comment

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

The three new additions are extremely useful, thanks for the contribution. Amendments to monitor_run.py lgtm.

@andrw3000 andrw3000 merged commit 80f4399 into andrw3000:main Sep 28, 2024
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