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

Report trending tags #578

Closed
theoreticalbts opened this issue Nov 17, 2016 · 1 comment
Closed

Report trending tags #578

theoreticalbts opened this issue Nov 17, 2016 · 1 comment

Comments

@theoreticalbts
Copy link
Contributor

theoreticalbts commented Nov 17, 2016

This ticket is to report trending in get_state as tags rather than categories.

Here is a summary of the changes:

  • Sort based on total_children_rshares2 rather than abs_rshares
  • Fix index bug where name is first sort key (should be second)
  • Delete trending from category_index
  • Add trending to tag_index
  • Create tags map which resolves things in tag_index [1]
  • Disable unused indexes

This is a slightly breaking change which needs some coordination with the web folks.

[1] Analogous to currently existing categories map which resolves things in category_index

theoreticalbts added a commit that referenced this issue Nov 17, 2016
mvandeberg pushed a commit that referenced this issue Nov 18, 2016
@mvandeberg
Copy link
Contributor

This was merged

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

No branches or pull requests

2 participants