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

Top Songs for an artist appear to be in random order #410

Closed
Harteex opened this issue Dec 13, 2023 · 9 comments · Fixed by #487
Closed

Top Songs for an artist appear to be in random order #410

Harteex opened this issue Dec 13, 2023 · 9 comments · Fixed by #487
Labels
bug Something isn't working

Comments

@Harteex
Copy link

Harteex commented Dec 13, 2023

Expected Behavior

Top Songs for an artist should be ordered in a sensible order, for example based on plays or at least recent plays.

Current Behavior

Top Songs are sorted in a way that doesn't make sense, as far as I can tell. It places songs with no plays at all higher up than those that have.
bild

Steps to Reproduce

  1. Click an artist and view Top Songs

Possible Solution (Not obligatory)

Context

Your Environment

  • Application version (e.g. v0.1.0) : 0.5.2 (feishin.vercel.app)
  • Operating System and version (e.g. Windows 10) : Windows 10
  • Server and version (e.g. Navidrome v0.48.0) : Jellyfin 10.8.9
  • Node version (if developing locally) :
@Harteex Harteex added the bug Something isn't working label Dec 13, 2023
@Optekah
Copy link

Optekah commented Dec 14, 2023

It's always been like this for me too, so it's not just you. Jellyfin and Navidrome both on Unraid.

On a somewhat related note, The "recent albums" on this same page as the "top songs" also seem to not display in anything other than a random order as well. Albums from 2023 are buried in the list while ones from the middle or even the beginning of an artists career can pop up at the very forefront.

@jeffvli
Copy link
Owner

jeffvli commented Dec 14, 2023

For Jellyfin the sort uses CommunityRating I believe, which in the case of no metadata will return a random sort. I'll probably just change this to use the PlayCount instead.

Navidrome should work properly, granted you have the lastfm integration enabled, since it pulls the top song list from there.

@iiPythonx
Copy link
Contributor

On a somewhat related note, The "recent albums" on this same page as the "top songs" also seem to not display in anything other than a random order as well. Albums from 2023 are buried in the list while ones from the middle or even the beginning of an artists career can pop up at the very forefront.

+1 for this, I can't say I'm having the track sorting issue but my albums on Navidrome are definitely in a random order.
According to the Navidrome UI:
image

And then Feishin:
image

@Optekah
Copy link

Optekah commented Dec 14, 2023

I personally have no use for the "top songs" and stuff like that haha. I would be super grateful if there was a way to just display all releases by an artist in grid format as soon as I click on the band name. Perhaps I could make a request post.

Navidrome should work properly, granted you have the lastfm integration enabled, since it pulls the top song list from there.

Ah this makes sense. Good to know, thanks!

@jeffvli
Copy link
Owner

jeffvli commented Dec 14, 2023

I would be super grateful if there was a way to just display all releases by an artist in grid format as soon as I click on the band name. Perhaps I could make a request post.

Someone already opened an issue for this (though I don't know which one it is off the top of my head), but it's already something I was planning on implementing.

Regarding the recent releases album ordering on Navidrome -- I just confirmed it as well. For some reason the _sort parameter is missing when performing the query which is why it's using a random order. I'll fix that soon.

@Optekah
Copy link

Optekah commented Dec 14, 2023

but it's already something I was planning on implementing

Fantastic news. Very thankful for all your hard work!

@cyberwolfie1
Copy link

but it's already something I was planning on implementing

Fantastic news. Very thankful for all your hard work!

I am also very happy about this planned enhancement :)

@iiPythonx
Copy link
Contributor

Regarding the recent releases album ordering on Navidrome -- I just confirmed it as well. For some reason the _sort parameter is missing when performing the query which is why it's using a random order. I'll fix that soon.

Are there still plans on this being fixed? Artist albums still show up in a random order when viewing their artist page with Navidrome.

@kgarner7
Copy link
Collaborator

kgarner7 commented Feb 3, 2024

Yes, see 24bf7ae. Sort is by year since that's supported for v0.48.0-current, eventually it can be release date when server features are available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants