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

Update example.config.yml #10

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

Conversation

nbarry202
Copy link

Added config options to go with suggested changes in timelapse.py
Updated interval to valid value

Added config options to go with suggested changes in timelapse.py
Updated interval to valid value

# Whether to create an animated gif after capturing the time-lapse images.
create_gif: False
create_gif: True
Copy link
Owner

Choose a reason for hiding this comment

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

I'd rather keep this default (and create_video) False for now, just because on any of the longer timelapses I've done, I found it took many minutes, sometimes hours, to create the gif or video on the Pi. So instead, I shut down the Pi, put the card into my desktop, then let the desktop crunch all the pictures together... usually about 100x faster :)

Reverting defaults
Copy link
Author

@nbarry202 nbarry202 left a comment

Choose a reason for hiding this comment

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

Hi Jeff - sure thing, I've reverted those changes.
Cheers
Nic


# Whether to create an animated gif after capturing the time-lapse images.
create_gif: False

# Whether to create an video after capturing the time-lapse images.
create_video: False

# Directory
directory: timelapse
Copy link
Owner

Choose a reason for hiding this comment

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

The 'directory' option isn't used anywhere, I don't think...

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