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

Use different algorithm for trending tags #7697

Merged
merged 1 commit into from
Jun 1, 2018
Merged

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Jun 1, 2018

Sort hashtags based on how different the currently observed usage frequency is to the expected (previously observed) usage frequency. Count uses by unique people instead of total volume within time frame. Threshold of 5.

Refs:

@ghost
Copy link

ghost commented Jun 1, 2018

How does this look like on small instances and starting instances? I'm afraid this won't work like desired until there are a minimum amount of toots with hashtags available.

A solution would be to hide trends (not disabling) until a usable number of (recent) toots+hashtags are available.

@Gargron
Copy link
Member Author

Gargron commented Jun 1, 2018

I have to separately rework the getting started column again to adjust for feedback about trends taking up too much space (actually, I get both feedback about how it's too much, and about how it doesn't display enough trends) but this algorithm unlike the previous one will not display tags that aren't actually being used, and so it may deliver empty results, in which case hiding the widget makes sense.

@ghost
Copy link

ghost commented Jun 1, 2018

@Gargron Ok, that sounds good. Although I would advise to hide poor results (as far as it's possible to measure that) as well. Imagine a new user or a new instance owner on a small/new instance. They will get a bad impression from Mastodon and the rest of the fediverse, even much worse than looking to the public timelines, cause trending hashtags are of course a subset of all available public toots.

@Gargron Gargron merged commit 6e6c0e9 into master Jun 1, 2018
@Gargron Gargron deleted the feature-improved-trends branch June 1, 2018 17:19
lawremipsum pushed a commit to lawremipsum/mspsocial-mastodon that referenced this pull request Jul 7, 2018
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

Successfully merging this pull request may close these issues.

1 participant