add property of puissance_nominale_grouped in opendata_stations.csv#38
add property of puissance_nominale_grouped in opendata_stations.csv#38tykayn wants to merge 3 commits intoJungle-Bus:masterfrom
Conversation
…ut folder in doc.
nlehuby
left a comment
nlehuby
left a comment
There was a problem hiding this comment.
Merci pour les modifications ❤️
| else: | ||
| # Si plusieurs puissances différentes, on prend la valeur la plus haute | ||
| station['attributes']['puissance_nominale_grouped'] = max(puissances) | ||
| errors.append({"station_id" : station_id, |
There was a problem hiding this comment.
c'est vraiment une erreur d'avoir des puissances nominales différentes sur les différents points de charge d'une station ? (c'est une vraie question, désolée si c'est une question bête 😋 )
There was a problem hiding this comment.
Très bonne question, d'un point de vue utilisateur la valeur la plus haute est la plus pertinente car les puissances d'un point de charge ne se cumulent pas, elles sont limitées au max parmi celles disponibles et qui peut le plus peut le moins.
Donc si quelqu'un a un véhicule qui prend de la haute puissance, il pourra cibler une station dont au moins un point de charge est capable de délivrer la plus haute en priorité pour savoir où charger le moins longtemps possible. Car souvent, quand on a une puissance max importante, ça veut dire que la station est moderne et qu'on aura plus de facilités à s'y charger, et que y'aura souvent plusieurs points de charge de cette haute puissance. On pourrait aussi garder l'info de l'énumération mais ça me semble plus pertinent d'avoir seulement le max.
On pourrait tenter de faire ensuite un mapping entre les tags des sockets et ceux de la puissance nominale, mais pour faire simple je pensais plutôt ajouter une info dont je connais l'avantage utilisateur et avec un résultat moins hasardeux.
Ce serait bien si les producteurs de données faisaient les choses dans le détail et pouvaient donner clairement les puissances de chaque socket mais bon, on n'y est pas encore et le csv ne permet pas d'avoir clairement l'info. :)
There was a problem hiding this comment.
d'ailleurs tu m'as donné une idée, on peut avoir une colonne avec la valeur de base et la valeur qui n'a que le max, comme ça les gens qui réutilisent auront le choix.
group_opendata_by_station.py
Outdated
| if len(puissances) == 0: | ||
| station['attributes']['puissance_nominale_grouped'] = None | ||
| elif len(set(puissances)) == 1: | ||
| station['attributes']['puissance_nominale_grouped'] = puissances[0] |
There was a problem hiding this comment.
est-ce que ce elif est vraiment utile ? on pourrait prendre le max dès que la liste est non vide, non ?
There was a problem hiding this comment.
tout à fait, j'ai mis à jour les enchaînements de conditions
README.md
Outdated
|
|
||
| Le fichier open data utilisé est le fichier consolidé des bornes de rechage pour véhicules électriques, publié sur [datagouv](https://www.data.gouv.fr/fr/datasets/fichier-consolide-des-bornes-de-recharge-pour-vehicules-electriques). Il s'agit d'un jeu de données qui regroupe l'ensemble des données produites par les différents acteurs territoriaux. | ||
|
|
||
| ## Installation |
There was a problem hiding this comment.
est-ce que tu peux déplacer les instructions d'installation à la fin du fichier readme ? je préfèrerais que le début reste lisible par un non codeur qui tomberait sur ce repo depuis Osmose ou le wiki.
There was a problem hiding this comment.
ok j'ai bougé les instructions d'install en fin de readme
README.md
Outdated
| ``` | ||
|
|
||
| **Note** : L'ID dans l'URL peut changer si le jeu de données est mis à jour. Vérifiez régulièrement la page du jeu de données pour obtenir l'URL la plus récente. | ||
|
|
There was a problem hiding this comment.
tant qu'à faire, ajouter un paragraphe "Exécuter le retraitement" avec python group_opendata_by_station.py
There was a problem hiding this comment.
ajouté une partie pour lancer le script
|
Sur le principe, ça me va, mais je n'ai pas trop les compétences métier pour avoir un avis éclairé sur la pertinence de la modif. |
|
il y a un problème métier :) autre élément : je pense que l'amélioration de la doc mériterait d'être dans un PR séparé qui pourrait alors être tout de suite mergé, de manière plus claire que inclus dans un PR dont le sujet est autre |
Co-authored-by: Noémie <noemie.lehuby@zaclys.net>
4 participants
Add the property puissance_nominale_grouped in the csv. The aim is to have this property used in the Osmose IRVE analysis and LibreChargeMap.
The property's value taken is the higher value if the value is an enumeration as it is the interesting data for EV owners this column gives, we can build something more complicated but this is a first step to get this info available in the output.