Menu

Quelques pistes d'améliorations

Olorin
2012-08-07
2012-09-03
  • Olorin

    Olorin - 2012-08-07

    Bonjour (je me permet le francais, a priori, vu le code ca ne devrait pas déranger)

    Tout d'abord un grand merci, enfin quelqu'un qui s'intéresse à la gestion de la musique sur les Dunes, en version multi-plateforme en plus, ca manquait.

    J'aimerais apporter quelques pistes de réflexions sur de futures améliorations.

    1) Tout d'abord, il manque un "paint.icons=no" dans les fichiers dune_folder.txt de plus bas niveau (ceux avec la tracklist) pour ne pas avoir une espèce de barre blanche masquant une partie des titres.

    2) Discodune ne gère pour l'instant pas les album double ou triple. Pour ma part ils sont classé de cette facon : Artist/Album/CDx/piste.flac
    Je pense que ce serait utile de gérer ce genre de chose.

    3) Pour les albums de plus de 100 titres (oui, je sais, ce n'est pas courant mais bon...)
    il y a un problème d'organisation, on se retrouve avec 01...10, 100...109,11,110 etc. (problème classique d'ordre alphabétique avec les chiffres). Mes fichiers sont pourtant bien nommés avec 3 chiffres (001 - titre.flac). Il faudrait soit compter le nombre de pistes et passer à une numérotation à 3 chiffres à partir de 100, soit garder le formattage du nom de fichier.

    4) Plus compliqué, ce serait bien de donner un peu plus de liberté au niveau de l'organisation des dossiers.
    Par exemple moi, pour des raisons de facilité de navigation, j'ai fait des répertoires alphabétiques dans lesquels je range les artistes (tous les artistes commençant par A dans le répertoire A, etc...) ce qui n'est pas géré. (bon, je me débrouille en faisant une passe sur chaque répertoire, d'ailleurs, voir point 5)
    Je pense qu'il serait pratique de pouvoir spécifier un schéma de rangement en paramètre, mais je ne sais pas si c'est facile à faire dans l'état actuel (je n'ai pas encore exploré le code, mon java étant un peu rouillé)

    5) Ne pas effacer la discothèque à chaque fois qu'on relance le soft, pour faire de l'incrémental, ne supprime que ce qui a changé (et avoir une option pour ne rien supprimer du tout, ca peut être pratique)

    6) pouvoir scanner plusieurs emplacements en même temps (cas typique, un utilisateur bordélique qui a une partie de sa musique sur un NAS, une partie sur le disque interne de la Dune, une partie sur le ou les PCs...)

    Voilà, c'est tout pour l'instant, j'espère que ces quelques remarques seront utiles. Bon courage et bonne continuation, ce soft peut aller loin (une fois internationalisé), l'organisation musicale manquait cruellement, il y a de la demande.

     
  • boblemar

    boblemar - 2012-08-08

    Bonjour,
    C'est vrai que j'ai laissé les commentaires en français pour l'instant.
    Je ne pense pas que cela gène grand monde pour l'instant vu l'intérêt suscité par mon projet...
    C'est vrai qu'à l'heure actuelle, c'est plus que rudimentaire et que l'approche Appli console / fichier de configs peut rebuter pas mal de monde, mais bon, j'ai fait ça d'abord pour moi et je me suis dit qu'étant donné le vide à ce niveau, quelques courrageux pourraient être intéressés.
    Pour le futur proche, je viens de mettre en place une approche multithread, préalable à une première version graphique. J'espère faire cette version rapidement.
    Pour le reste...

    1/ Je vais regarder. Ca ne me dit plus rien. Pas très compliqué...

    2/ Certes, c'est une known issue... Je le sais. J'avais réfléchi à des pistes, mais ce n'était pas urgent pour moi. Je m'en accomode en supprimant 1 niveau de répertoire : Album/CD1/ devient Album CD1/, mais bon l'idée n'est pas d'adapter le stockage au logiciel, mais l'inverse. Bonne remarque.

    3/ Celle-là, bravo ! Je regarderai rapidement, surement pas très complique ! (Je n'ai pas eu le cas...)

    4/ Bonne idée. Pas prioritaire pour moi, mais bonne idée.

    5/ Pour l'instant, je ne vois pas bien comment faire sans reprendre en profondeur l'outil. Personnellement, je ne regénère pas la discothèque tous les jours, donc je laisse tourner le temps que ça prend, et au bout d'un moment, un long moment, c'est bon ! Mais je comprends qu'on puisse vouloir ripper un CD que l'on vient d'acheter et le voir apparaître rapidement.

    6/ J'ai tout sur mon NAS ;), mais c'est une demande que j'avais remontée pour yaDIS, donc ce n'est pas illégitime. Le moteur pourrait s'en accomoder facilement. Je regarde.

    Dans tous les cas, merci de tes remarques. Ca fait plaisir de voir que son travail peut servir à d'autres.
    Aller loin, je ne sais pas. Pour l'instant je suis motivé, mais le temps me manque, mais au pire, si j'arrive à livrer quelque chose d'utilisable pour l'utilisateur lambda, d'autres prendront le relais.
    Mais avant cela, il faut que j'avance un peu et que je nettoie un peu le code (mon java n'est pas non plus en très bonne forme : le semaine, c'est plutôt c# ou vb.net, html et javascript... que j'ai le plaisir de développer professionnellement...)

     
  • Olorin

    Olorin - 2012-08-08

    Je ne suis pas sûr que le coté CLI/fichiers de config soit si rebutant. Bien sûr une interface graphique est plus rassurant pour monsieur tout-le-monde, mais regarde YAMJ pour les Popcorn Hour par exemple, il est assez utilisé malgré la ligne de commande (il y a d'ailleurs peut-être des idées à reprendre sur ce projet).

    Pour l’intérêt suscité par le projet, il faut dire que tu n'as pas l'air d'avoir fait beaucoup de pub pour l'instant, je suis tombé ici un peu par hasard. Une fois le code un peu nettoyé, un petit mot sur HDLand et sur HCFR te rapporterais surement quelques utilisateurs (voire de l'aide ?), et une fois internationalisé, pourquoi pas sur mpcclub.

    Pour ma part je vais essayer de mettre les mains dans le code, voir si je peux apporter quelque chose, ca me changera de mon C quotidien (va falloir que je me souvienne comment ca marche la prog objet...).

    (tiens une idée supplémentaire en passant, pour plus tard, rendre l'interface skinnable, ca ne devrait pas être trop compliqué, il "suffit" de mettre les fichiers dune_folder de base en ressource externe plutôt que de les avoir en dur dans le code)

     
  • boblemar

    boblemar - 2012-08-17

    Je viens de publier un V2 avec UI. C'est un peu plus joli, et la création de la bibliothèque est multithreadée. Je n'ai pas Xthreadé la publication car je ne suis pas sur que ça a un intérêt.
    Concernant tes demandes... elles sont dans un coin de ma tête, mais pas intégrées, à part les n° de pistes au dela de 100 (mais pas test;)).
    Concernant le paint-icon, je n'ai pas de soucis chez moi, peut-être utilisais-tu une ancienne version. Au pire, je regarderai.
    Je pense que dès que j'aurai un peu de temps, j'intégrerai la possibilité de donner plusieurs dossiers source. J'avais commencé, mais ça faisait casser pas mal de trucs alors j'ai préféré publier cette version.
    Concernant la pub, effectivement, je n'ai rien fait. Je voulais avoir une version "montrable"...

     
    • Olorin

      Olorin - 2012-08-18

      J'ai vu que tu as bien avancé cette semaine, j'ai suivi les commits au fur et à mesure

      C'est bizarre, même avec les dernières versions j'ai toujours une espece d'icone blanche (je suppose que c'est l'icone de base representant un fichier musical) qui cache une partie des titres si je ne met pas l'option paint_icon à no. Ca dépend peut-être du modèle de dune ou du firmware (j'ai une HD Max avec le dernier firmware beta). L'option est à rajouter dans MusicFormat.java

      Pour la publication multi-threadée, je suis d'accord avec toi, pas certains que ce soit utile.

      Pour les numéros de piste au delà de 100, je confirme, ca fonctionne.

      Pour info, j'ai réussi à adapter le programme à mon arborescence, en ajoutant notamment la gestion des albums multi-disque. Ca se fait assez facilement en ajoutant une couche entre Record et MusicFormat (par contre vu que j'ai soit du flac soit du mp3 mais que je ne gardes pas les 2 formats, je n'ai pas testé les interactions lorsqu'il y a plusieurs formats.)

      Un truc que j'ai remarqué hier que je n'ai pas encore exploré, j'ai certains fichiers dont l'extension comporte des majuscules (.Mp3 ou .MP3 par exemple). Apparemment ils ne sont pas pris en compte. Je n'ai pas encore regardé comment se fait la détection de fichiers, mais il doit sûrement être possible de faire cette détection de facon insensible à la casse.

      Dernière chose, j'ai vu que tu as enlevé une grande partie du feedback je trouve ca dommage, en tant que linuxien à l'ancienne j'ai toujours une préférence pour la ligne de commande, et là on n'a plus d'info de l'avancement, j'aimerais bien que tu gardes un mode verbose (quitte à en faire une option supplémentaire à passer en argument)

       
      • boblemar

        boblemar - 2012-09-03

        Je viens de publier un petit correctif. Effectivement, après avoir mis à jour mon firmware, j'ai eu le problème de l'icône peinte sur le libellé des pistes.
        J'ai remis un peu de traces par la même occasion (en mode verbose)...

         

Log in to post a comment.

MongoDB Logo MongoDB