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 configuration example for retain modes #3589

Merged
merged 4 commits into from
Aug 13, 2022

Conversation

ZuluWhiskey
Copy link
Contributor

Reading this page of the documentation on its own didn't help me understand how to configure my Frigate instance. But when I found #2447, I was able to understand how to add it to my configuration. I've added the example given in that discussion to help future readers of the page.

I myself am still not 100% confident with the configuration setup - If possible it may be worth someone helping to also add the all option into the configuration example if that's possible? eg. all retained for 1 day, motion for 7 and active_objects for 14?
Would it worth adding object retention examples here too? eg.

record:
  events:
    retain:
      default: 14
      mode: all
      objects:
        person: 30
        car: 14

And my PR is for the global configuration, I can't seem to find an example of camera specific retention unfortunately so haven't added that.

Reading the documentation on its own didn't help me but when I found blakeblackshear#2447 I was able to understand how to add this to my configuration. I've added the example given in that discussion to help future readers of the page.
@netlify
Copy link

netlify bot commented Aug 4, 2022

Deploy Preview for frigate-docs ready!

Name Link
🔨 Latest commit 1828841
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/62f78ca8fc19a70008989a70
😎 Deploy Preview https://deploy-preview-3589--frigate-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

docs/docs/configuration/record.md Outdated Show resolved Hide resolved
Added suggested changes and have also added wording beneath the example mentioning the configuration can be added on a per camera basis.
Have also built on the example to add object specific retentions timings - Not sure if it would be preferred to have it all within one example to not complicate things?

Let me know your thoughts
Created Object Specific Retention header
@blakeblackshear blakeblackshear merged commit 7a1215d into blakeblackshear:master Aug 13, 2022
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.

None yet

3 participants