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

Poor Performance on Mobile Device #46

Closed
6 tasks done
seanwu1105 opened this issue Aug 10, 2020 · 0 comments · Fixed by #85
Closed
6 tasks done

Poor Performance on Mobile Device #46

seanwu1105 opened this issue Aug 10, 2020 · 0 comments · Fixed by #85
Assignees
Labels
code Improvements to code quality or project architecture fix/feature verified The fix has been verified by QA priority:high High priority uiux UI or UX update
Milestone

Comments

@seanwu1105
Copy link
Contributor

seanwu1105 commented Aug 10, 2020

Extremely poor performance on mobile devices. Need time to investigate the root cause.

  • It is highly possible that the Filesystem on the Android device is not fast enough to rapidly query multiple times.
    • Possible fix: merge the information tuples into a large json file.
    • Possible fix: use sqlite instead.
  • Media data use base64 encoding to pass around instead of file object with source URI.

Roadmap

  • Use <img /> instead of <ion-img> for better performance.
  • Merge the information tuples into a large json file. This require new "equal" class to check the equality. Greatly add complexity.
  • Call Storage.refresh$() only on initialization.
    • Remove calling Storage.refresh$() on Storage.add$().
    • Remove calling Storage.refresh$() on Storage.remove$(). This require new "equal" class to check the equality. Greatly add complexity.
  • Generate the thumbnail of the media asset on captured.
@seanwu1105 seanwu1105 added uiux UI or UX update code Improvements to code quality or project architecture priority:high High priority labels Aug 10, 2020
@tammyyang tammyyang added this to the beta3 milestone Aug 26, 2020
@seanwu1105 seanwu1105 linked a pull request Sep 2, 2020 that will close this issue
@scott-dt scott-dt added the fix/feature verified The fix has been verified by QA label Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Improvements to code quality or project architecture fix/feature verified The fix has been verified by QA priority:high High priority uiux UI or UX update
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants