Results 15 comments of Guillaume Seguin

Hmm you probably have a track artist or title which has some foreign character which cannot be encoded in Unicode ?

> Can you add a test for this? Done! I haven't been sure how to add reviewers explicitly to the PR (I assumed I'd need a well documented issue to...

Hm that remaining PR failure seems to be spurious? 🤔

Is there a way to jog the CI to rerun the failed checks?

Is there something else needed to get this merged? (I cannot add other reviewers it seems, if that's what's needed)

> Thanks @iXce for the suggestion. We could also add an boolean attribute to `Metric` which all metrics descend from, called `compute_per_rank` and give it to the constructor of a...

> @iXce thanks for the RFC! We also wanted to be able to override metrics decorators responsible for reducing data: #1288. So, you could override it and do a no-op...

rebased to the latest :) @uranusjr anything I should do to get this merged? I encounter needs for this almost on a daily basis (a very good example is the...