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: Added forced visual capture script #95

Merged
merged 3 commits into from
Jul 24, 2022

Conversation

MateoLostanlen
Copy link
Member

As discussed offline we need a function to capture an image from each camera mainly for debugging purposes. However it is dangerous to do what was done in the PR #92 we don't want to interfere with the runner. This PR proposes to remove the modification from the previous PR and create an independent script

@MateoLostanlen MateoLostanlen requested review from a team July 24, 2022 17:01
@MateoLostanlen MateoLostanlen self-assigned this Jul 24, 2022
@MateoLostanlen MateoLostanlen added the invalid This doesn't seem right label Jul 24, 2022
@codecov
Copy link

codecov bot commented Jul 24, 2022

Codecov Report

Merging #95 (8594a25) into develop (ef6348e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #95   +/-   ##
========================================
  Coverage    62.63%   62.63%           
========================================
  Files            5        5           
  Lines          182      182           
========================================
  Hits           114      114           
  Misses          68       68           
Flag Coverage Δ
unittests 62.63% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us.

@MateoLostanlen MateoLostanlen merged commit 5ed2437 into pyronear:develop Jul 24, 2022
@MateoLostanlen MateoLostanlen deleted the feature/capture branch July 24, 2022 17:21
@frgfm
Copy link
Member

frgfm commented Jul 24, 2022

Quick question: why did you add the label "invalid" ? 😅

@MateoLostanlen
Copy link
Member Author

Hi @frgfm, the current capture method was "invalid". But maybe I'm not using the tag properly

@frgfm
Copy link
Member

frgfm commented Jul 27, 2022

capture method? what are we talking about? the camera?

the tag "invalid" is usually used on "invalid" issues for instance 😅 (meaning that the raised concern / issue is irrelevant or non applicable)

@frgfm frgfm added ext: src type: new feature and removed invalid This doesn't seem right labels Aug 7, 2022
@frgfm frgfm added this to the 0.2.0 milestone Aug 7, 2022
@frgfm frgfm changed the title Feature/capture feat: Added forced visual capture script Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants