Skip to content

test-pr - #5

Open
TeresaHo wants to merge 1 commit into
gabyvon725:masterfrom
TeresaHo:test-pr
Open

test-pr#5
TeresaHo wants to merge 1 commit into
gabyvon725:masterfrom
TeresaHo:test-pr

Conversation

@TeresaHo

Copy link
Copy Markdown

Done by 何家蓁

import sys
from datetime import datetime

import cv2

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

這邊不需要 import cv2哦



# Read until video is completed
while(self.cap.isOpened()):

@gabyvon725 gabyvon725 Mar 15, 2018

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

(建議) Line33 可以省略括號

# Read until video is completed
while(self.cap.isOpened()):
ret, frame = self.cap.read()
if ret == True:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Line 35 可以寫成 if ret:,因為通常較少用 == 來比較物件跟 True/False boolean

cv2.imshow('Frame',frame)
self.frames.append(frame_r)
if cv2.waitKey(25) & 0xFF == ord('q'):
break

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Line 39 40 這邊可以不寫哦。 因為目的就是要讓它處理完,不存在中途需要quit的問題。

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