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

[iOS] 실시간으로 Coordinate 기록 시 위치 필터링 구현 #303

Merged
merged 10 commits into from
Dec 11, 2023

Conversation

yoondj98
Copy link
Collaborator

❗ 배경

작업 배경에 대한 설명을 작성합니다.
Issue에 대한 링크를 첨부합니다.

여정을 기록 시 모든 위치 값을 기록하는 것을 방지하는 로직을 구현했습니다.

🔧 작업 내역

작업한 내용들을 나열합니다.
간결하게 리스트 업하고, 자세한 설명은 아래 리뷰 노트에서 합니다.

  • 여정을 기록 시 5초마다 위치가 5m초과 50m미만의 거리를 이동하였을 경우에만 기록이 되도록 구현했습니다.
  • 5초 내로 기록이 안되었다면 다음 5초 후를 기준으로 위치 비교를 하도록 구현했습니다.

🧪 테스트 방법

동작을 테스트할 수 있는 방법을 설명합니다.
앱 실행 방법일 수 있고, 유닛 테스트 실행 방법일 수 있습니다.

📝 리뷰 노트

작업 내역에 대한 자세한 설명을 작성합니다.

위치 보정에 대한 고민

5초, 5m< distance <50m의 기준은 임의로 정한 것이기에 정확도가 높은, 모든 상황을 고려한 필터링은 아닙니다.

  • 위치가 5초동안 0~5m 이동 후 그 다음 50m를 넘어간다면 추후엔 저장이 안되는 이슈가 발생할 수 있습니다.
  • 이 부분에 대한 추가적인 보정 고민이 필요합니다.
  • 이 점을 고려해서 Kalman 필터를 추후 학습 후 적용을 해야할 듯 합니다.

📸 스크린샷

작업한 내용에 대한 스크린샷, 영상 등을 첨부합니다.

@yoondj98 yoondj98 added 🐞 버그 픽스 발견된 버그 (목록, 해결 여부) ✨ 신규 기능 신규 기능 개발 labels Dec 11, 2023
@yoondj98 yoondj98 added this to the 🏃‍♂️ 여정 기록 milestone Dec 11, 2023
@yoondj98 yoondj98 self-assigned this Dec 11, 2023
Copy link
Member

@SwiftyJunnos SwiftyJunnos left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
현재 상태에서 할 수 있는 가장 합리적인 방법이네요 👍

이후에 추가적인 알고리즘 적용하면 좋을 것 같아요
프로젝트 끝나기 전에 간단한 개선정도는 고민해봐도 좋을 것 같습니다.

@SwiftyJunnos SwiftyJunnos merged commit 5ab1859 into iOS/release Dec 11, 2023
33 checks passed
@SwiftyJunnos SwiftyJunnos deleted the iOS/task/HomeButton-Bug-Fix branch December 11, 2023 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 신규 기능 신규 기능 개발 🐞 버그 픽스 발견된 버그 (목록, 해결 여부)
Projects
Status: ✅ 완료
Development

Successfully merging this pull request may close these issues.

3 participants