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
Count more reactions when considering the highest rated comment #2190
Comments
I'd keep it simple. When it counts, thumbs up and downs are already the majority and I doubt that considering the rest of the emojis would make any difference. |
I agree with @bfred-it although ❤️ seems like a good candidate but would probably require checking for people who voted both 👍 and ❤️ to make it fair. |
Before anyone thinks of implementing this, I need to see cases where this actually makes a difference, because I have doubts. Links please. @lubien what you’re saying isn’t always possible, we don’t have the full list of users who reacted, just the first 10 for each reaction |
From a quick search: Here's one without a top post at the moment. But with the proposed changes this would have been a top post: Here's one where if we count all reactions the top post would have been a different one: |
That being said, I realize now that counting multiple reactions does introduce more complexity in how we count. For example, what if 1 user adds multiple positive reactions to a post? Do we count all reaction? Or does each user only get 1 vote? Is the added complexity worth a "better" top post? And how to determine which method of adding/subtracting makes the result more valid? |
True, but also you can see a lot of user overlap in those reactions. There are still 7 unique users.
👍
Ok, maybe it is. Just remove the users that appear duplicates in the list, keeping in mind that we can't deduplicate users that don't appear in the list (because it's limited to 10 usernames) |
Or maybe convert reactions to points. Example:
|
@TiagoDanin I feel that the problem with that is that we can't make a scoring system with different values clear to the user. With your proposed scoring system it's not immediately obvious why a post with If each one is worth only 1 point it's easier to immediately see and understand why a post is a top post. Also, if your proposed scoring system somehow does become clear to the users then we will be implicitly training users of (Refined) GitHub to react in certain ways to maximize a post. And if certain reactions are "worth more" that can eventually make the entire reactions system less truthful. |
FYI: This already happens if we allow anything in this issue. I can add 4 votes by reacting with 👍 😄 🎉 ❤️ in busy posts with more than 10 reactions each (i.e. where you can't deduplicate my points) |
Oh wow, I didn't know that Github only names 10 users max per reaction. In that case I recommend not implementing this feature until we can get all the names so that we can deduplicate. |
We're not gonna make API calls for each comment in a thread only to get a slightly better "highest rated comment" though 😅 We can deduplicate the first 10 and just accept that 1 user = up to 4 votes, if we make this change. |
Haha, fair point! It's an acceptable compromise :) |
I don't care about this feature so much, but take this as a case: My proposal: find the comment with most reactions (as for me, I would choose from these reactions 👍 🎉 ❤️ 🚀) and mark it as the highest-rated comment. In the example above, the first comment gets 31 🎉, which exceeds the second comment's 14 👍, so the first comment should be marked as the highest-rated comment. |
What about showing the 'comment with most reactions' instead of the 'highest rated'? IMHO it is legit if a user wants to add more than a single point. Precisely, I use it to make a difference between 'this is good/interesting' and 'I'd love to have this implemented/merged'. By the same token, I think that refined-github should not decide which reactions are 'good' or 'bad'. A 'thumbs up' is as legit/informative as a 'thumb down', and not more relevant than 'eyes'. This is a 'controversial' example, where there are 'mixed reactions': microsoft/vscode#17996 |
Picking “the best one” is as far as RGH can go. Ideally you’d want some UI to “sort by reactions,” like StackOverflow would show answers, but that doesn’t belong to this extension. We should strive to “be helpful in most cases” rather than trying to cover all cases. This feature just needs to be changed to count 👍 🎉 ❤️ instead of just 👍 |
If that's the case, then could I try this issue? |
It's all yours :) |
Because of #2108 we can see the most helpful comment (thanks @lubien!). At the moment that only counts
👍
as a positive and👎
as a negative. But there are more reactions that can be used to indicate a post is helpful or not. How about we also count those?I've left the ambiguous
👀
and (the slightly less ambiguous)🚀
out.The text was updated successfully, but these errors were encountered: