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: add README and useful bash scripts #1

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Helias
Copy link

@Helias Helias commented Sep 6, 2022

Copy link
Member

@Benehiko Benehiko left a comment

Choose a reason for hiding this comment

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

Thank you for adding these to the repository. I think it would be really helpful to other users :)

download.sh Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated
## download.sh

This script uses `ffmpeg` to download via RTSP the video and audio from your camera saving it in the MP4 format, the name will start with the `date` of the day (ex. `2022-08-26--22-00-00-capture-0000.mp4`) and a file will be recorded and saved every `$TIME` seconds.
The name of the files starts with a date to make it easier the management of the files, like removing easily all the records of a specific month, day or year.
Copy link
Member

Choose a reason for hiding this comment

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

How does this work, does the stream get downloaded until a certain point? and could we do this "indefinitely" until the script is stopped?

Some questions I think that should be answered:

  1. How do i stop the download of the stream?
  2. Limits of downloading the RTSP stream?
  3. Expected file sizes? also stream quality and network requirements / limitations.

Copy link
Author

Choose a reason for hiding this comment

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

I don't know exactly the answers to these questions.
To stop the stream the user can just type "Ctrl+C", for the rest I still need to make experiments and activate my camera at home, which will take a long time.

download.sh Show resolved Hide resolved
clean_records.sh Outdated
@@ -0,0 +1,2 @@
LAST_MONTH=$(date --date='-1 month' +'%Y-%m')
rm "tmp/$LAST_MONTH-"*
Copy link
Member

Choose a reason for hiding this comment

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

maybe we could move this script inside the downloads script as a flag instead since the user would probably not always use the tmp directory.

Copy link
Author

Choose a reason for hiding this comment

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

I would use a separate script because using crontab a user can in parallel run this only script to delete the old files

Copy link
Member

Choose a reason for hiding this comment

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

we can achieve the same thing using a flag with the download script, but i think your approach is better since it cannot be accidental when you literally run "clean_records". But I think you still need to be able to delete stuff in a different folder and thus need to accept an argument for a directory

Copy link
Author

Choose a reason for hiding this comment

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

done

README.md Show resolved Hide resolved
Helias and others added 3 commits September 9, 2022 22:00
Co-authored-by: Alano Terblanche <18033717+Benehiko@users.noreply.github.com>
Co-authored-by: Alano Terblanche <18033717+Benehiko@users.noreply.github.com>
@Helias
Copy link
Author

Helias commented Sep 11, 2022

script and README updated

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.

2 participants