Correction du Calcul temps de trajet

Parce que je ne sais même pas ce que c’est qu’un PR ? :man_shrugging:
Je suis pas dev, les repo et tout, c’est pas mon truc en vrai :blush:

Maintenant, si quelqu’un avec de réelles compétences (en code et en git) se sent de proposer l’update du plugin, qu’il n’hésite pas :+1: :wink:

J’ai noté pour regarder ajd.
J’avais regardé rapidement hier après-midi mais je recevais des http 403 depuis jeedom donc je pensais que le problème était la et je n’avais pas eu le temps de regarder plus

Edit: pas eu le temps du coup, grosse journée.
Ça sera pour demain.

2 « J'aime »

Si tu peux, refais peut-être un check sur l’adéquation du résultat de la somme des crossTime avec la valeur de total_route_time.
Depuis que j’utilise ma version avec total_route_time le résultat a toujours été cohérent par rapport à la réalité et à ce que j’avais avant, j’ai assez peu de doute du coup, mais encore une fois, j’ai pas été regarder dans la doc de l’API pour confirmer ce point.

1 « J'aime »

Merci ! J’ai effectué la modif et ca fonctionne chez moi également !

1 « J'aime »

Un petit retour: j’ai inclus les changements de mon coté, j’en ai profité pour faire un peu de maintenance dans l’écran de config pour être plus au goût de jeedom v4, quelques fixes et optimisations et j’ai inclus le support de la config geoloc faite dans le core comme option supplémentaire (en plus des commandes existantes et de l’encodage manuel).

Visuellement, on passe de ceci:


à

trouvez les 7 différences :joy:

Je vais encore faire quelques tests de tout ca avant de proposer le changement en beta, probablement demain.

3 « J'aime »

Ah, j’ai bien fait de pas faire de RP sur le repo :grin:
Du coup on va même avoir un plugin tout propre !

Merci @Mips :+1:

1 « J'aime »

Update de ce matin (on peut le voir à la couleur des screens :joy:):


c’est facile ici, il n’y a qu’une différence :wink:

PR fait:

@Loic, vu que tu es le dernier à avoir fait des changements sur le plugin, si tu as le temps de valider mon PR en beta qu’on puisse valider que cela résout le problème pour tout le monde?
Merci

Salut,
Le soucis c’est que je connais pas du tout le plugin je peux donc pas dire si ton PR est bon ou pas… Donc je le valide en te faisant confiance, par contre que ce soit clair je ne fais AUCUN SAV sur ce plugin (j’en ai deja trop a gerer)

Ça roule. On laisse en bêta et j’attends les retours pendant quelques jours.
S’il y a des soucis je gère.

Je te fais signe s’il y a un nouveau pr à valider ou si on peut passer en stable.

Merci !

Super merci

Merci à tous.

Merci pour cette béta qui refait fonctionner ce plugin.
Je ne sais pas si il y aurait un intérêt à trier les trois trajets suivant le temps de parcours.

C’est pas le cas par défaut et fait par waze? j’ai toujours eu le trajet le plus court (en temps) en premier et ainsi de suite.

Capture d’écran du 2021-10-18 11-07-31 Capture d’écran du 2021-10-18 11-08-05

:thinking:

bonjour,

Cela semble fonctionner pour les temps dans les commandes, test demain en présentiel :slight_smile:

par contre widget vide, mais je ne l’utilise pas :stuck_out_tongue:
image

Curieux ca, quelle version de jeedom?
Aucune personnalisation activée? (css ou autre)
les trajets sont bien configuré pour être affichés? voir config sur l’équipement

Hello Mips,

Par curiosité j’ai réactivé le plugin que j’avais désactivé depuis plus d’un an mais j’ai pas mieux que @skillix

image

Pas de personnalisation (css ou autres) d’activées
Jeedom en 4.1.27 Debian Buster

Trajets configurés

Log:

[2021-10-18 16:21:26][INFO] : Refresh routes pour: Trajet Travail
[2021-10-18 16:21:26][ERROR] : Internal Error
[2021-10-18 16:26:22][INFO] : Refresh routes pour: Trajet Travail
[2021-10-18 16:26:22][ERROR] : Internal Error
[2021-10-18 16:26:33][INFO] : Refresh routes pour: Trajet Travail
[2021-10-18 16:26:33][ERROR] : Internal Error
[2021-10-18 16:27:19][INFO] : Refresh routes pour: Trajet Travail
[2021-10-18 16:27:20][ERROR] : Internal Error
1 « J'aime »

décoche Amérique du nord !

c’est con c’est coché par défaut et on fais pas gaffe ^^

2 « J'aime »