Skip to content

Commit fa55af8

Browse files
authored
Merge pull request #111 from guerda/feat_genre_top
feat(genres): skip n/a as genre, list 25 top genres
2 parents 6686827 + 96870a4 commit fa55af8

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

app.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ async def get_genre_count(
139139
request: Request,
140140
beets_statistics: Annotated[BeetsStatistics, Depends(get_beets_statistics)],
141141
):
142-
genres = beets_statistics.get_genre_count(limit=20)
142+
genres = beets_statistics.get_genre_count(limit=30)
143143
genre_list = []
144144
count_list = []
145145
for genre in genres[0]:

beetsstatistics.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -121,13 +121,11 @@ def get_genre_count(self, limit: int = 0):
121121
cursor = self.get_db_connection().cursor()
122122
res = cursor.execute(
123123
"""select
124-
case
125-
when a.genre = '' then "n/a"
126-
else ifnull(case when instr(a.genre, ';') then substr(a.genre, 0, instr(a.genre, ';')) else a.genre end , "n/a")
127-
end as genre,
124+
ifnull(case when instr(a.genre, ';') then substr(a.genre, 0, instr(a.genre, ';')) else a.genre end , "n/a") as genre,
128125
count(1) as count
129126
from
130127
albums a
128+
where a.genre != ''
131129
group by
132130
a.genre
133131
order by

0 commit comments

Comments
 (0)