-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Rating System #42
Comments
The thing is that this is not the way LRCLIB was designed from beginning. It was designed for finding a single, best exact match for your song based on metadata (name, artist, album, duration). If the lyrics is somehow inaccurate or out of sync, you can just directly submit a correction. Your newly submitted lyrics would essentially overwrite the older one for that metadata. Think of it as Wikipedia but for lyrics. |
Sometimes for a song there might be like 8 listings that are all identical. And maybe I have a better version that applies to all 8 (since all 8 are literally identical, just different albums (like a Deluxe version, a compilation album, etc.) Or some other detail that doesnt affect the song length and timestamps, its just a different release), then im not sure what to do. |
a malicious actor could just submit blank files / corrupt lyrics one of the ways to solve this could be to have some sort of voting but i think this would a bit too much one other soln would be to set a pref in client, like only DL lyrics from official sources and avoid user submitted lyrics |
Aren't they all user submitted? And if they're from some other source then a user had to submitted them to LRCLIB. |
Ahh okay. |
So I don't know where to submit ideas/feature requests for lrclib, but I want to suggest that there could a rating system so that people could rate (maybe out of 5 stars) lyrics so that lrcget could factor in the rating and preferably pick the highest rating when searching for lyrics.
This would likely require an account system for lrclib which you could sign in with lrcget. I know this would likely require some work to implement but I think this would be beneficial.
The text was updated successfully, but these errors were encountered: