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

안드로이드 간단한 예제 추가 #5

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

Conversation

SwimingKim
Copy link

Pull Request - 이미지 분석 예제

관련된 해당 이슈의 종류는 무엇인가요? Feature

Feature

이 PR에 대해 명료하게 요약 해주세요.

비트맵을 불러와서 분석해주는 간단한 예제입니다.

이 pr 관련해서 다른 팀원들이 알아야하는 내용이 있나요?

권한 추가, dependency 추가, 레이아웃 수정, 클래스 코드 작성을 했습니다.

실제로 앱 빌드해서 테스트 되었나요?

  • [v] 기본적인 디바이스(삼성, LG 최신폰)에서 UI 테스트 완료
  • 서버와 연동 테스트 완료
  • 영문화 대응 확인 완료
  • 과거 버전 대응 완료

QA 시에 테스트 해야할 부분은?

@skfo763
Copy link
Member

skfo763 commented Jan 23, 2021

우선 오랜 기간 관리되지 않던 리포지토리에 관심 가져주시고 기여해주셔서 감사합니다!
코드리뷰 후 문제 없으면 바로 머지하도록 하겠습니다


class MainActivity : AppCompatActivity() {

private var mClassifier: Classifier? = null
Copy link
Member

Choose a reason for hiding this comment

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

onCreate 메소드에서 초기화되는 property이므로 lateinit var 로 선언해도 괜찮을 것 같은데 어떻게 생각하시나요?

Copy link
Member

@skfo763 skfo763 left a comment

Choose a reason for hiding this comment

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

코멘트 하나 달았습니다. 확인해보시고 피드백 부탁드립니다!

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.

4 participants