Grâce à la version 1.3 de Duniter et au travail d’@elois, nous disposons désormais de cet outil mais pour Ğ1-Test :
3 Likes
Ops, j’ai mal géré ma configuration Nginx, le service revient dans quelques minutes …
edit : voilà c’est mieux !
1 Like
Currency-monit était déjà disponible sur g1-test depuis deux semaines via mon nœud a l’adresse : http://g1-test-monit.elois.org
Ah, désolé, j’ai loupé l’information.
Sinon, je viens de te faire une PR pour accepter quelques changements :
- affichage de la distance « OK » ou « KO »
- liens aussi pour l’UID présent dans chaque certification
- possibilité de paramétrer l’URL de wotex, car le g1-test-monit renvoie par défaut sur wotex.cgeek.fr, qui tourne sur Ğ1
Du coup, cliquer sur un pseudo mène à mon 1er module effectif : wotex ! Bientôt dans les bacs numériques 
1 Like
Bon je viens de merger et de pull sur mon noeud de dev et j’ai des KO partout pour la distance c’est curieux 
EDIT : problème réglé, le souci se situe ici : https://github.com/duniter/duniter-currency-monit/blob/dev-module/routes/willMembers.js#L218
Je ne récupérait pas le champ membersCount du bloc courant car je n’en avait pas besoin avant ta PR, je viens donc d’ajouter ce champ à la ligne 15 et ça fonctionne 
EDIT2: ce que je ne comprend pas @cgeek c’est comment tu a fait pour ne pas rencontrer se problème en vue de ta capture d’écran ?
C’est très curieux oui, car maintenant j’ai effectivement que des KO.
Bon, tu as spotté le bug, c’est le principal ! 
Merci d’avoir accepté cette PR, je me sens mieux 
1 Like
Par contre il semble qu’un de mes commit est cassé le fonctionnement en module.
Tout fonctionne nickel en passant par le run.js mais si je plug mon dossier de dev sur un nœud duniter impossible de faire fonctionne le plugin, je vais investiger 
EDIT : C’est pas très verbeux je n’ai aucune piste, @cgeek peut tu zieuter vite fait mon commit ? la cause te sautera peut-être aux yeux : https://github.com/duniter/duniter-currency-monit/commit/70cd2b2f75b328619227116d8495e4a367776984
ben fallait quand même que je prenne le temps de comprendre ce que fait ton code avant de merger 
J’ai une erreur dès le mode développement :
events.js:160
throw er; // Unhandled 'error' event
^
Error: SQLITE_CANTOPEN: unable to open database file
at Error (native)
Je pense que ça vient de cette ligne, car je n’ai pas cette BDD :
Pour moi, program.mdb vaut undefined. Ce doit être le cas dans n’importe quel nœud quand on branche le moniteur en mode module.
1 Like
En fait, tu peux remplacer :
home+'/.config/duniter/'+program.mdb
Par :
duniterServer.home
Je sais, c’est écrit nulle part, on a l’impression que je sors ça de mon chapeau. Mais avec un outil de debug, on peut voir tout ce que transporte duniterServer comme propriété, c’est là que je l’ai vu (et je me doutais que je le fournissais quelque part ce home).
1 Like
Même en supprimant la bdd et en relançant sa fonctionne nickel chez moi. En principe il créer automatiquement la bdd si elle n’existe pas. Le problème que j’ai ne viens donc pas de là c’est encore autre chose, mais du coup j’ai 2 problèmes au lieu d’un…
Oui je pense que mon problème de départ viens effectivement de là. Je vais utiliser duniterServer.home à la place merci beaucoup pour ton investigation !
EDIT : hé oui c’est ça, qui me manque, un vrai outils de débug, vas falloir que je finisse par m’y mettre 
1 Like
Et voilà release 0.2.4 fonctionnelle avec les nouvelles fonctionnalités de @cgeek, j’ai également rétabli la page membersCount
Pour l’installer sur votre nœud :
duniter plug https://github.com/duniter/duniter-currency-monit/archive/module-0.2.4.tar.gz
J’ai fait la mise à jour sur http://g1-test-monit.elois.org 
Il ne manque donc plus que la page pubkeyBalance… c’est un gros chantier et je ne suis pas sûr d’y arriver avant la deadline du 1er juillet (date à laquelle le protocole aura changer), il y aura donc une petite interruption du service pubkeyBalance car je préfère bien faire que de mal faire et pour cela j’ai besoin de prendre mon temps !
2 Likes
Bon, il doit rester un bug sur la distance : même les nouveaux arrivants sans signature sont à « distance OK » :
edit : voilà j’ai corrigé, je me suis fait avoir par l’indentation. Hésites pas à bien vérifier, mais déjà j’ai des KO sur les nouveaux qui n’ont pas de certifications.
mince croisement j’étais dessus en même temps 
tu avait juste placé le code au mauvais endroit, j’ai déjà commit et ça fonctionne bien sur le noeud de dev
EDIT : et en même temps je viens d’optimiser car inutile de calculer dSen a chaque fois, sa valeur est la même pour toutes les identités
Oui et puis s’il y a des doublons, on recalcule inutilement aussi.
Bref je te fais confiance sur ce coup là, tu connais ton algo mieux que moi
1 Like
exactement c’est pourquoi j’ai placé ton code après l’exclusion des doublons finalement
OK, j’attendrais ta release pour réinstaller mes modules alors.
J’ai vérifier manuellement sur wotex quelques identités avec une seule certif et elles respecte toutes bien la règle de distance, cela est du au fait que le groupe de membre référent bloquants qui à un moment avait frôlé les 20% est redescendu, j’ai identifier environ 13 membres référents quasi-systématiquement bloquants qui n’ont probablement certifier que vers l’extérieur, mais par rapport aux 103 membres référents actuels ça ne fait que 12% donc ce n’est pas bloquant
Beaucoup de certifs internes sont passés ces 2 dernières semaines, il y a vraiment eu un mouvement de resserrement de la toile, peut-être suite a mon appel justement