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.

Sign up for GitHub

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

Jump to bottom

Feat/topn from db #44

Merged
merged 4 commits into from
Sep 4, 2019
Merged

Feat/topn from db #44

merged 4 commits into from
Sep 4, 2019

Conversation

tukeJonny
Copy link
Contributor

  • グラフデータを間引くさいのtopn処理に用いる、チームのID一覧をDBから取得するように
  • それに伴って不要になった処理や定義の削除

fixes #43

@tukeJonny tukeJonny added the bug Something isn't working label Sep 4, 2019
@tukeJonny tukeJonny self-assigned this Sep 4, 2019
@tukeJonny
Copy link
Contributor Author

passed().get だと通常の場合でも例外になっちゃうので修正

@chibiegg chibiegg merged commit 8d44460 into master Sep 4, 2019
@chibiegg chibiegg deleted the feat/topn-from-db branch September 4, 2019 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chibiegg chibiegg chibiegg approved these changes

Assignees

@tukeJonny tukeJonny

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

グラフ表示の際のtopn処理でfinished_atの考慮
2 participants
@tukeJonny @chibiegg