adafruit / adafruit/Adafruit-VC0706-Serial-Camera-Library

No "Stop taking pictures" function

Open
#11 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
C++
Stars
139
Forks
108
PR merge metrics
No merged PRs in 30d

Description

Although there is a function that does send the **Stop taking pictures** command (`boolean resumeVideo(void)`) it is not a very user friendly way of resetting the picture. Also, the name of the function `boolean takePicture(void)` indicates that a **new** picture is taken, which is only the case for the first picture.

I would suggest that the **Stop taking pictures** command is sent inside the `boolean takePicture(void)` function before a new picture is taken.

Contributor guide

No contributing guide indexed for this repository

Research direction

The relevant entry points are boolean resumeVideo(void) and boolean takePicture(void); read their current command flow to understand how reset and capture interact. Verify that invoking takePicture() sends the Stop taking pictures command before starting a new picture, and check any existing validation path for the library if one is available.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.