Je m'excuse si j'ai pu te froisser dans mes propos et je n'avais aucune intention d'enfoncer quoi que ce soit...
Bonjour, je me permet juste de donner un avis en tant que professionnel du développement. Je suis tout à fait d'accord avec ce que dis sarakha63.sarakha63 a écrit : ↑17 août 2017, 20:51Ben mon retour n'apportera rien de plus que ce que j'ai dis avant.
Utiliser une lib toutes faites ne veut pas dire la coller au milieu d'un environnement sans se poser de questions.
...
Non je t'assure thanaus lorsque tu développes pour un environnement multicomposant comme une domotique et qui plus est quand ça va être utilisé par des utilisateurs finaux il faut pas se contenter de se reposer sur une lib toutes faites.
Une lib toutes faites c'est bien l'adapter et faire en sorte qu'elle s'intègre bien dans l'environnement cible c'est mieux.
D'autant plus quand ton plugin arrive alors que ya déjà 3 plugins existants qui eux fonctionnement très bien sans de marcher dessus.
J'ai été très surpris d'un tel retour aussi rapide de votre part que ce soit toi ou @magicgg91 pour un simple log E_NOTICE.sarakha63 a écrit : ↑17 août 2017, 20:51Ben mon retour n'apportera rien de plus que ce que j'ai dis avant.
Utiliser une lib toutes faites ne veut pas dire la coller au milieu d'un environnement sans se poser de questions.
Peut importe la version que tu ais la dernière ou pas.
Nous de notre côté on a adapté la lib pour justement éviter ça. Car on avait trois plugins.
Mais la.de ce que je comprends tu as pris la lib et la mets dans un environnement domotique et pars du principe c'est la lib a jour donc pas de soucis...
Non je t'assure thanaus lorsque tu développes pour un environnement multicomposant comme une domotique et qui plus est quand ça va être utilisé par des utilisateurs finaux il faut pas se contenter de se reposer sur une lib toutes faites.
Une lib toutes faites c'est bien l'adapter et faire en sorte qu'elle s'intègre bien dans l'environnement cible c'est mieux.
D'autant plus quand ton plugin arrive alors que ya déjà 3 plugins existants qui eux fonctionnement très bien sans de marcher dessus.
D'ailleurs tu sais que ta raison évoqué pour faire un plugin commun payant est pas valable. "Éviter de rentrer des tokens à trois endroits". Pour info chacun des 3 autres plugins vérifie si son petit frère existe pour récupérer les identifiants chez lautre et éviter au user de le retaper.
Après tu as le droit de faire un plugin c'est pas le soucis. Chacun fait ce qu'il veut. Mais évite de donner une raison fausse.
Quoiqu'il en soit pour ton soucis c'est donc bien dans la lib qu'il faut que tu gères les chargements déjà faits ou pas. Car comme je le.dis la lib fourni par netatmo c'est bien. Mais from scratch elle est pas spécialement bien foutu non plus notamment sur le chargement des classes et variables dans un environnement complexe autre que celui de la lib pur et dur
Je suis entièrement d'accord avec toi et sarakha63 sur le fait d'utiliser et/ou modifier une API pour l'intégrer au mieux dans un environnement de production.Sli3H3llFiR3 a écrit : ↑17 août 2017, 21:34Bonjour, je me permet juste de donner un avis en tant que professionnel du développement. Je suis tout à fait d'accord avec ce que dis sarakha63.sarakha63 a écrit : ↑17 août 2017, 20:51Ben mon retour n'apportera rien de plus que ce que j'ai dis avant.
Utiliser une lib toutes faites ne veut pas dire la coller au milieu d'un environnement sans se poser de questions.
...
Non je t'assure thanaus lorsque tu développes pour un environnement multicomposant comme une domotique et qui plus est quand ça va être utilisé par des utilisateurs finaux il faut pas se contenter de se reposer sur une lib toutes faites.
Une lib toutes faites c'est bien l'adapter et faire en sorte qu'elle s'intègre bien dans l'environnement cible c'est mieux.
D'autant plus quand ton plugin arrive alors que ya déjà 3 plugins existants qui eux fonctionnement très bien sans de marcher dessus.
Je vais généraliser pour faire simple. Une librairie issue d'un constructeur permet d'utiliser en général 2 élément fondamentaux. (lorsque c'est bien fait xD:) :
- Des interfaces : En langages non développeur, on peux dire que ce sont des "points d'entrées" permettant d'appeler des fonctions.
- Des implémentations : C'est ce que font les fonctions.
Normalement (et là encore, lorsque c'est bien fait), une API doit proposer une architecture (plus ou moins) similaire afin de permettre d'utiliser les points d'entrées (interfaces) communes, tout en laissant 2 choix :
- Utiliser les implémentations originales du produit, adaptées au produit dans son environnement initial.
- Utiliser des implémentations "maison" adaptées à l'environnement spécifique.
Sans pour autant connaître les adaptations effectuées par les équipes Jeedom, il se peux qu'il ai du faire face à cette problématique => créer un "adapter" afin de permettre le bon fonctionnement.
Après il est tout a fait possible d'utiliser (via les interfaces), des implémentations native en mixant avec des implémentations spécifiques. Dans beaucoup de cas, il n'est pas nécessaire de faire du from scratch. De plus si la lib est dans un langage permettant le polymorphisme (comme PHP 7, Java, C#), il est même possible d'étendre la fonction principale en ajoutant des subtilité.
Cordialement
Je pense que tu as tout à fait raison de n'être sur le forum que pour mettre fin à un débat et atténuer des tensions. Et tu as bien entendu toute la légitimé pour le faire.sarakha63 a écrit : ↑17 août 2017, 21:57Mais aucun soucis. Je reproche pas.le plugin je te dis juste que la raison mise en avant n'est pas réel. Mais pas grave.
Concernant le.copinage dont tu fais allusion.
Je ne suis plus le forum pour plein de raisons personnelles que j'ai évoquées. Mais bien évidemment sur des canaux de communication externes ça parle des problématiques soulevés. Et si je juge que je peux intervenir pour éclairer les choses je le fais. D'où mes interventions ponctuelles mais généralement qui mettent fin à un débat et/ou apporte un éclaircissement.
Je préfère 100 fois ce mode de fonctionnement ou on me remonte un éventuel soucis/interrogation à traiter et intervenir. Que de passer ma vie à son péril sur le forum comme je le faisais ya 2 ans.
Ce mode permet de pas passer trop de temps dessus mais d'être au courant quand même.
Donc oui on est peu present mais on est toujours là et généralement au courant. Et si besoin d'éclaircir un point on intervient.
Donc ce n'est pas du copinage mais.un mode de fonctionnement bien confortable pour nous.
Ceci étant dit OUI je suis très proche de Magicc et lunarok et bien d'autres d'ailleurs mais là n'est pas la question. Car les interventions ne sont rarement du copinage mais généralement nécessaire ou apportant un vrai éclaircissement et dans 90% des cas ça atténue une tension. Comme hier par exemple
Il n'y a aucun souci pour ma part....DenisPac a écrit : ↑17 août 2017, 19:59Euh je suis désolé que ma log soit le sujet de débats aussi vifs. Ça n'était pas mon intention.
Il doit sans doute y avoir une manière de trouver une solution technique qui soit satisfaisante pour toutes les parties.
Et en effet le retour de docteur @saraka63 est attendu pour soigner tout ce malentendu
Bonjour @thanaus, pas de problème je veux bien tester ta nouvelle version.thanaus a écrit :Il n'y a aucun souci pour ma part....DenisPac a écrit : ↑17 août 2017, 19:59Euh je suis désolé que ma log soit le sujet de débats aussi vifs. Ça n'était pas mon intention.
Il doit sans doute y avoir une manière de trouver une solution technique qui soit satisfaisante pour toutes les parties.
Et en effet le retour de docteur @saraka63 est attendu pour soigner tout ce malentendu
Est-ce que tu serais d'accord pour mettre à jour ton plugin manuellement avec un .zip que je te fournirai demain soir ? Ce .zip comportera la correction afin que ce log ne soit plus présent.
Si tu es d'accord et que le log n’apparaît plus, je mettrais à jour le market avec cette correction.
Merci à toi
Je viens de t'envoyer en MP un lien pour récupérer la dernière version du plugin avec une correction pour que ce log d'information ne soit plus présent.
@thanaus, la correction est fonctionnelle et peut être déployée en beta, puis stable.thanaus a écrit : ↑18 août 2017, 12:11Je viens de t'envoyer en MP un lien pour récupérer la dernière version du plugin avec une correction pour que ce log d'information ne soit plus présent.
Si ok pour toi (je n'ai plus pour ma part ces logs), je basculerais la version en beta, puis en stable
En parallèle, je vais proposer à l'équipe Jeedom de mettre à jour les autres plugins en lien avec netatmo pour utiliser la dernière version de l'API Netatmo. Cela permettra ainsi que tout autre développeur voulant utiliser cet API dans un futur plus ou moins proche ne soit indirectement victime de ce log...
Très bonne journée à tous
Merci beaucoup pour ton soutien des la première heure. Cela fait très plaisir !cyrilphoenix a écrit : ↑18 août 2017, 15:33Et bah ya d'l'ambiance ici......
Courage thanaus ton plugin est au TOP
Envoyé de mon Mi-4c en utilisant Tapatalk
Il n'y a pas actuellement la commande pour récupérer la version du firmware mais je peux sans problème l'ajouter même si je ne vois pas exactement ce que tu souhaites en faire.
Bonjour
Cette nouvelle version sera en beta des ce soir et en stable lundi.
Pas de souci pour ma part, même si j'ai toujours une erreur 500 qui me reste (c'est en cherchant la cause de cette erreur que je suis tombé sur les logs supplémentaires de ce plugin).
Si tout bien compris, tu veux être prévenu qu'un nouveau firmware a été mis à jour sur ta station même si c'est fait en automatique sans aucune intervention de ta part. C'est exact ?Jeandhom a écrit : ↑18 août 2017, 19:31Il n'y a plus la possibilité de mettre à jour manuellement le firmware de la station météo. Il est uniquement poussé sur les stations par les serveurs de Netatmo. Le dernier firmware est disponible depuis le 12 avril dernier.
J'en ai marre de regarder régulièrement (depuis le 12 avril) si le firmware de ma station est à jour.
Je préfèrerai laisser Jeedom opérer, via un scénario.
J'ai fait, plusieurs fois, la demande sur le plugin officiel que je possède, mais sans succès.
Utilisateurs parcourant ce forum : Aucun utilisateur inscrit et 9 invités
