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

Async tweet load #13

Merged
merged 2 commits into from
Oct 14, 2019
Merged

Async tweet load #13

merged 2 commits into from
Oct 14, 2019

Conversation

Yicong-Huang
Copy link
Contributor

@Yicong-Huang Yicong-Huang commented Oct 14, 2019

This PR is to fix #12 for 1) and 2) step.

  • 1. when initialized, load tweet count aggregated by date.
  • 2. when selecting a time range, request for the exact tweet id and location with in the time range.
  • 3. select tweets based on map range
  • 4. frontend buffer to cache tweets for duplicate or overlap selection

@Yicong-Huang Yicong-Huang merged commit da43e7c into master Oct 14, 2019
@Yicong-Huang Yicong-Huang deleted the async-tweet-load branch October 20, 2019 19:37
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.

Tweet Load Memory Issue
2 participants