Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

[PLUGIN RHYTHMBOX] synchronisation de vos notes dans vos fichiers MP3

masterbox

Alors, je dis amen a toutes tes remarques, faut croire que j'aurais dû analyser un peu plus le plugin dont je suis parti, mais bon j'avais fait au plus court... Enfin toutes sauf une remarque tout de même, celle qui concerne la condition d'import.
En effet, si je test si le nombre de count est un 0, parfois la note ne s'importera pas alors qu'il y en a bien une.
Par contre je serait partisan de faire deux cas différents :

1) le rating existe, on importe rien
2) le rating est à 0, alors on importe la note, et on ajoute le compteur à celui qui est dans rhythmbox (pour eventuellement le remettre dans le fichier apres), car cela veut dire qu'on a écouté la musique depuis que le fichier a été déplacé par exemple, ou encore que les notes viennent d'un autre lecteur dans lequel on a lu ce fichier, et donc le compte correspond bien aussi à la somme des 2.

Qu'en penses-tu ?

Mais de manière générale, la façon la plus sûre est de synchroniser à l'importation du fichier (du coup tout est à 0) et puis à chaque lecture / modification de la note. Cela provoque par contre beaucoup d'écritures des fichiers...

Sinon voici le git ( https://github.com/masterbox/saveTAGRating ) auquel j'avais ajouté tes premières recommendations. Mais si tu as déjà une version à jour tu n'as qu'à l'écraser.

Vysserk3

Bon alors,
J'ai commité et maintenant je pense que la base est saine à savoir :
- factorisation au maximum de code et modularité, ca permettra de développer facilement une fonction sans toucher au reste du code qui est indépendant du format audio par exemple
- j'ai amélioré aussi la fonction de sauvegarde et de restauration pour éviter de faire une écriture supplémentaire ou un commit supplémentaire si on n'en a pas besoin
- j'ai essayé de commenter un maximum le code pour que ce soit le plus lisible possible

Voilà, reste à prendre en charge les autres formats (malheureusement, d'après ce que j'ai lu il n'y a pas de standard pour le ogg ou le flac) et à faire les gui threads 😉

Si on veut exécuter quelque chose à la lecture ou à la modification d'un rating, je pense qu'il faudra passer par dbus pour ça.

Vysserk3

Bon pour les GUI threads, c'est fait, j'ai mis en IDLE la priorité pour pas que ca gêne d'autres choses. Ca a l'air relativement rapide et fluide (j'ai testé juste sur une centaine de fichiers, faudrait voir avec genre 5000-10 000 fichiers).

masterbox

je regarde tout ça dans la semaine dès que j'ai un peu de temps, merci pour ton aide inestimable :-)

je testerai avec quelques milliers de fichiers.

Vysserk3

Par contre, je viens de me rendre compte d'un truc, si je suis ce qu'indiquait samfuzz, on devrait changer en fait le tagging même pour les mp3....:

2.1.1 MP3

MP3 values MUST be stored in a TXXX frame with the Description
set to the specified identifier and the Text set to the string
representation of the value. The Description SHOULD be in
CamelCase as specified in the following sections, e.g.
FMPS_Rating.

2.1.2 VorbisComments

Any file supporting VorbisComments (Vorbis, FLAC, Theora, Speex)
MUST use a comment with the Key set to the specified identifier
and the Value set to the string representation of the value. The
Key SHOULD be in all upper-case, e.g. FMPS_RATING.

En gros au lieu de POPM et PCNT, il faut utiliser un TXXX....mais après pas sûr que ce soit lisible par XBMC ou autres lecteurs, mais bon ce serait plutôt à ces lecteurs de s'adapter. Au pire on peut stocker les trois types de frame....

Pour les vorbis/flac, il faudrait s'inspirer du tagging (clé:valeur) de Quod Libet (cf http://code.google.com/p/quodlibet/wiki/Specs_VorbisComments)
Tout ceci devrait être possible via mutagen uniquement et pas trop dur à changer maintenant que le code est bien modulaire.
À suivre....

masterbox

Ok pour la spec, implémenter un TXXX est très simple je te le confirme avec mutagen (plus simple même que le POPM)

Par contre je ne souhaite pas abandonner le POPM pour une raison simple, beaucoup de players l'utilise encore sans implémenter cette spec free desktop. On peut très bien mettre les 2 tags dans le fichier en écriture. En lecture soit on donne la priorité au tag TXXX (plus simple dans un premier temps au moins), soit on gère plus finement les conflits potentiels (2 notes différentes dans les 2 tag, en provenance de 2 sources différentes)

Pour OGG et Flac, la spec de Free Desktop est assez clair il me semble.

Je vais implémenter le tag TXXX dans un premier temps si tu veux bien, histoire de me familiariser avec le plugin modularisé 😉
Ensuite on s'occupera du OGG et du FLAC.

La synchro "temps réelle" n'est pas vraiment prioritaire.

Vysserk3

Oui je te laisse faire ça, tu as raison, j'étais en train d'éditer mon post, pour l'instant, je pense qu'on devrait stocker et le POPM/PCNT et le TXXX, qu'en penses-tu ? Ou alors tu vérifies avant sur XBMC s'il reconnait un TXXX avec une description FPMS (pas sûr car la spec est récente...).
Donc le mieux c'est de marquer que les POPM/PCNT seront déprécié un jour ou l'autre, mais on les laisse, suffira de commenter les bonnes lignes le jour où ce sera nécessaire....

masterbox

Exactement.

Vysserk3

Ok bon j'essaye de faire le Ogg/FLAC en attendant, apparemment c'est plus simple que pour les tags ID3, on verra bien 😉

Vysserk3

Bon c'est fait pour le support ogg/oga/flac , beaucoup plus simple que l'ID3 car les tags sont dans un dictionnaire python 😉
J'ai respecté les FPMS_RATING et FPMS_PLAYCOUNT seulement (juste les valeurs canonique, pas de truc suivant l'utilisateur ou selon l'algorithme).

masterbox

Je viens de rajouter le tag FMPS_Rating pour le MP3.
Par contre je crois que tu as fais le dislexique sur FMPS en le transformant parfois en FPMS, me trompe-je ? 😉 En plus sur les playcount c'est parfois l'un, parfois l'autre...
Je me permets de corriger pour respecter la spec. Dis moi si tout est ok !
J'ai rajouter une breve
A part ça super clean le refactoring, beaucoup plus clair, ça me réconcilierait presque avec Python 🙂

Vysserk3

Oui j'étais justement en train de corriger cette erreur (entre autres) 😉 J'essaye de me sortir du conflit du merge qu'il y a maintenant...

masterbox

arg les merges beurk.
Pourtant j'ai essayé de faire vite ;-)

Vysserk3

Il y a moyen que tu te connectes sur jabber ? Ca évitera de polluer le forum je pense 😉
J'ai réussi en partant de zéro car j'ai pas trop l'habitude de git, mais je sais que normalement, on ne doit pas éditer les mêmes lignes si on reste dans le même fichier (il y aura forcément des conflits qu'il faudra résoudre à la main, pas de merge auto), mais là j'ai du faire une fausse manip, faudra que je relise la doc de git encore.
Sinon, je vais faire quelques petits ajustements sur ce que tu as fait si tu veux bien, par exemple au lieu de divisé par 5, il vaut mieux multiplier par 0.2 et ainsi la conversion vers float se fait toute seule (int * float = float) (il aurait fallu diviser par 5.0 si on avait voulu garder la division mais sans devoir mettre float() devant
Aussi, au lieu de var.__str__() un str(var) me semble plus lisible (derrière str(var) appelle la méthode de l'objet var de toute façon).

masterbox

Demain on en parle sur Jabber alors. Tant pis pour nos fans.

samfuzz

je viens de tester le greffon avec ma bibliothèque (notes taggués avec les specs FMPS via gmusicbrowser)

2 petites remarques :

1 - je suis sur lucid et le path vers le répertoire plugins est :
~/.gnome2/rhythmbox/plugins/saveTAGRating
j'ai du modifier la ligne :

self.pluginrootpath = path.expanduser("~/.local/share/rhythmbox/plugins/saveTAGRating/")

pour avoir accès au menu contextuel

2 - le champ TXXX:FMPS_Rating pour les id3 est sensible à la casse (voir les specs)
j'ai du remplacer pour id3v2 "FMPS_RATING" par "FMPS_Rating" pour pouvoir importer les notes dans rhythmbox
(a noter vorbiscomment : FMPS_RATING:
ID3v2 : FMPS_Rating)
peut-être faudrait-il que le greffon soit insensible à la casse en lecture uniquement (importation).

bonne surprise, mes fichiers sont taggés de 1 à 10, l'importation s'est faite sans problème : (FMPS : 0.9 ---> importé 5 pour rhythmbox) comme le veut la spec FMPS :

For instance, if a track has a rating of 0.9 and an application
only shows ratings using five star icons in full-star increments,
this would be rounded within the application to five stars.
Howeer, if the player also shows the rating numerically, the
application would display 4.5 in the numeric field instead of the
same 5 shown in the star icons, thus more accurately reflecting
the user's set rating.

par contre quoi faire lorsque FMPS_Rating = 0 ?

sinon, est-il possible d'ajouter les formats Musepack (APEv2), et M4a, ça semble possible avec mutagen,j'ai essayé de le faire moi même mais comme je débute (en fait je pars de zéro) j'ai quelques difficultés, enfin c'est l'occasion d'apprendre un tout petit peu un language et la programmation.

concernant XBMC j'ai trouvé ça :
http://wiki.xbmc.org/index.php?title=Preparing_your_Music
XBMC sait lire le champ TXXX:RATING, pour les id3 , par contre je ne sais pas quels sont les lecteurs qui l'utilisent, idéalement si XBMC pouvait supporter les spec FMPS ça serait plus simple

Vysserk3

Merci pour le retour samfuzz
1) Effectivement, l'endroit des plugins dans ~/.gnome2 est celui indiqué dans la doc sur les plugins de rhythmbox, cela dit ca devrait aussi marcher avec l'autre répertoire théoriquement. Je vais changer ça. En fait je devrais changer ça en prenant le répertoire courant de là où est exécuté le plugin (si c'est possible).

2) Bien vu pour le FMPS_RATING et FMPS_Rating, j'avais pas fais gaffe à ça....et je crois que masterbox l'avait bien fait au début en plus et comme un imbécile, j'ai du modifier ça dans la précipitation. Par contre, sur un simple script python, j'ai ouvert un fichier avec un tag enregistré avec des majuscules, et la valeur de la clé apparaît en minuscule...mais c'est peut être juste l'affichage qui fait ça, c'est pas forcément comme ça derrière.

Par contre c'est curieux que le 0.9 se soit transformé en 5. Car normalement ce qu'on fait c'est 5*0.9 c'est à dire 4.5 converti en entier, donc 4....

Aussi un choix (arbitraire) que j'ai pris c'est de dire que si on doit importer des valeurs nulles, on ne fait rien en fait (par sécurité). On peut toujours changer ça. Pour la sauvegarde par contre, si une valeur de la db est nulle, on efface le tag du fichier.

Pour les autres formats de fichiers, ca doit être possible tout dépend de quel type de tag on peut mettre, si c'est de l'ID3 ou du vorbiscomment, ca devrait être très simple à rajouter.

Pour XBMC, il faudrait leur demander de prendre en compte les tags de freedesktop 😉 Ca doit pas être bien compliqué à modifer de leur côté 😉

edit :
-C'est corrigé pour les deux premiers points.
-Concernant l'insensibilité à la casse, faut voir.
- Pour le 0.9 en fait c'est normal

masterbox

Pour le 0.9, on reste en float pour rentrer dans la DB Rythmbox, donc Rythmbox fait bien l'arrondi 🙂

Vysserk3

samfuzz >> Voilà j'ai ajouté le support pour le mp4/m4a, mais il faudrait tester, car sur un fichier de test de mutagen ca ne marche pas (erreur unicode au moment du save() ...)
Pour les fichiers .ape, je ne les vois pas dans Rhythmbox donc je peux pas tester (il faut peut être installer un truc pour lire ce format).

masterbox

Moi j'ai un peu cherché pour la progress bar... et suis tombé sur tes messages 😉

J'ai trouvé plusieurs plugins qui en affiche une, j'analyse tout ça...

« Page précédente Page suivante »