Currency-monit disponible sur Ğ1-Test!

modules
ğ1-test

#1

Grâce à la version 1.3 de Duniter et au travail d’@elois, nous disposons désormais de cet outil mais pour Ğ1-Test :

https://g1-test.cgeek.fr/modules/currency-monit/


#2

Ops, j’ai mal géré ma configuration Nginx, le service revient dans quelques minutes …

edit : voilà c’est mieux !


#3

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


#4

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 :slight_smile:


#5

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 :confused:

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 :grinning:

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 ?


#6

C’est très curieux oui, car maintenant j’ai effectivement que des KO.

Bon, tu as spotté le bug, c’est le principal ! :smiley:

Merci d’avoir accepté cette PR, je me sens mieux :slight_smile:


#7

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 :sweat_smile:

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


#8

ben fallait quand même que je prenne le temps de comprendre ce que fait ton code avant de merger :wink:


#9

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 :

https://github.com/duniter/duniter-currency-monit/blob/70cd2b2f75b328619227116d8495e4a367776984/lib/main.js#L16


#10

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.


#11

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).


#12

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 :stuck_out_tongue:


#13

Et voilà release 0.2.4 fonctionnelle avec les nouvelles fonctionnalités de @cgeek, j’ai également rétabli la page membersCount :slight_smile:

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 :wink:

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 !


#14

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.


#15

mince croisement j’étais dessus en même temps :stuck_out_tongue:
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 :wink:


#16

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 :slight_smile:


#17

exactement c’est pourquoi j’ai placé ton code après l’exclusion des doublons finalement :wink:


#18

OK, j’attendrais ta release pour réinstaller mes modules alors.


#19

it’s work :blush:

https://github.com/duniter/duniter-currency-monit/archive/module-0.2.6.tar.gz


#20

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 :wink:

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 :slight_smile: