johan/consolidate-ipe-data #5

Merged
johan.le.baut merged 2 commits from johan/consolidate-ipe-data into master 2022-10-14 13:56:32 +02:00
No description provided.
johan.le.baut added 11 commits 2022-04-13 14:37:51 +02:00
felix.baylac reviewed 2022-04-13 15:11:42 +02:00
@ -0,0 +20,4 @@
echo "[+] Ingesting IPE data from ${fullIpePath}"
echo ""
cat > "${tmpSql}" <<EOF
Member

CF. #6

CF. https://forge.aquilenet.fr/Aquilenet/Axione-IPE-Viewer/issues/6
Member

Note: c'est juste pour te mettre au courant. Cette PR est déjà assez grosse comme ça, on fera ça dans une autre.

Note: c'est juste pour te mettre au courant. Cette PR est déjà assez grosse comme ça, on fera ça dans une autre.
johan.le.baut marked this conversation as resolved
@ -5,3 +5,3 @@
nativeBuildInputs = [ pkgs.sqlite pkgs.cargo pkgs.rustc pkgs.poetry ];
shellHook = ''
LD_LIBRARY_PATH=${pkgs.libspatialite}/lib:LD_LIBRARY_PATH
export LD_LIBRARY_PATH=${pkgs.libspatialite}/lib:$LD_LIBRARY_PATH
Member

facepalm

*facepalm*
johan.le.baut marked this conversation as resolved
@ -1,2 +1,5 @@
[DB]
path = /path/to/ipe.sqlite
axione_ipe_path = /path/to/ipe.sqlite
Member

Je préfèrerais qu'on ne garde qu'une seule DB SQLite pour toutes les données. Ça facilite le remplacement des données lors des mises a jour.

Je préfèrerais qu'on ne garde qu'une seule DB SQLite pour toutes les données. Ça facilite le remplacement des données lors des mises a jour.
Member

Résolu.

Résolu.
Member

Résolu.

Résolu.
felix.baylac marked this conversation as resolved
@ -0,0 +12,4 @@
nc=narrow_coordinates
c = self.https_conn
req = "/souscription/gps-batiments.cgi?etape=gps_batiments&lat1=%f&lat2=%f&lon1=%f&lon2=%f" % (nc['swy'],nc['ney'],nc['swx'],nc['nex'])
req = req.replace(" ", "%20")
Member

Il vaudrait mieux utiliser urllib.parse.quote()

>>> import urllib.parse
>>> urllib.parse.quote("hello world")
'hello%20world'
Il vaudrait mieux utiliser `urllib.parse.quote()` ``` >>> import urllib.parse >>> urllib.parse.quote("hello world") 'hello%20world' ```
Member

Solved

Solved
johan.le.baut marked this conversation as resolved
@ -0,0 +1 @@
# from .cursor import *
Member

Fichier vide, à supprimer.

Fichier vide, à supprimer.
Member

Facepalm, c'est l'init.

*Facepalm*, c'est l'init.
johan.le.baut marked this conversation as resolved
webapp/main.py Outdated
@ -28,3 +44,3 @@
valid_args = True
processed_args = {}
for k in ['swx', 'swy', 'nex', 'ney']:
for k in ["swx", "swy", "nex", "ney", "centerlat", "centerlng"]:
Member

Il vaudrait mieux calculer centerlat et centerlng côté backend plutôt de faire confiance au client pour nous envoyer des trucs pas absurdes par rapport au points sud oust et nord est.

On a déjà tout ce qu'il faut:

centerlng = (swx + nex) / 2
centerlat = (swy + ney) / 2

Note: je serai plutôt pour les noms de coordonées coordonées x et y. IE. centerx, centery. Perso, je me perd tout le temps avec les trucs à base de lat/long.

Il vaudrait mieux calculer `centerlat` et `centerlng` côté backend plutôt de faire confiance au client pour nous envoyer des trucs pas absurdes par rapport au points sud oust et nord est. On a déjà tout ce qu'il faut: ```python centerlng = (swx + nex) / 2 centerlat = (swy + ney) / 2 ``` Note: je serai plutôt pour les noms de coordonées coordonées x et y. IE. centerx, centery. Perso, je me perd tout le temps avec les trucs à base de lat/long.
johan.le.baut marked this conversation as resolved
webapp/main.py Outdated
@ -69,0 +58,4 @@
# nex=processed_args['nex'],
# ney=processed_args['ney'],
# )
narrow_coordinates=AreaCoordinates(
Member

Je comprend pas trop ce truc.

La bounding box sw/ne est trop large pour Liazo?

Il y a une raison particulière pour ne pas utiliser la bounding box du client ici?

Je comprend pas trop ce truc. La bounding box sw/ne est trop large pour Liazo? Il y a une raison particulière pour ne pas utiliser la bounding box du client ici?
Author
Owner

oui c'était trop large sinon

oui c'était trop large sinon
johan.le.baut marked this conversation as resolved
@ -2,3 +72,3 @@
let markers = [];
function initMap(btn) {
// Init map position/zoom. Potentially using what's in the URL search string.
Member

Nit:

On dirait que ton editeur a massacré l'indentation qu'on avait. Ça crée pas mal de diffs artificielles. Si c'est facile de ton côté, ça serait cool de dégager ces diffs pour render les "vraies" plus facile a lire.

Nit: On dirait que ton editeur a massacré l'indentation qu'on avait. Ça crée pas mal de diffs artificielles. Si c'est facile de ton côté, ça serait cool de dégager ces diffs pour render les "vraies" plus facile a lire.
johan.le.baut marked this conversation as resolved
@ -1,118 +1,236 @@
const minZoomForRequest = 17;
Member

Trop de "fausses" diff et Gitea n'est apparemment pas aussi bon que GitHub pour les afficher. J'ai probablement raté des vrais trucs nouveaux.

Trop de "fausses" diff et Gitea n'est apparemment pas aussi bon que GitHub pour les afficher. J'ai probablement raté des vrais trucs nouveaux.
johan.le.baut marked this conversation as resolved
@ -98,0 +202,4 @@
const ne = bounds.getNorthEast();
const btn = document.getElementById("btn-load-elig-data");
waitBtn(btn);
const reqUri = encodeURI(`eligdata?swx=${sw.lng}&swy=${sw.lat}&nex=${ne.lng}&ney=${ne.lat}&centerlat=${mc.lat}&centerlng=${mc.lng}`);
Member

Pareil: enlever centerlat et centerlng.

Pareil: enlever centerlat et centerlng.
johan.le.baut marked this conversation as resolved
Member

Comme d'hab: tu peux ignorer les trucs préfixés par Nit: si t'as envie.

Comme d'hab: tu peux ignorer les trucs préfixés par `Nit:` si t'as envie.
felix.baylac requested changes 2022-04-13 15:25:36 +02:00
felix.baylac left a comment
Member

J'ai un gros doute niveau UX.

On ne récupère les données que du centre de la carte et non plus de toute la carte affichée.

Par ex:

Capture d'écran montrant le problème

Hors, dans ce cas-là, le reste du village pour lequel il n'y a pas de pins est bien dans la base de données ARCEP.

Je pense que ça peut perdre pas mal de monde en faisant penser que seul le centre de la carte affichée est couverte.

Je pense qu'il vaudrait mieux afficher toutes les données qu'on peut (ie. Arcep et Axione) pour toute la bounding box de la carte du client.

Puis, dans les tooltip, ajouter un bouton pour récupérer les données d'éligibilité Liazo pour un batiment précis à la demande de l'utilisateur.

Techniquement, derrière, on ferait une requête à Liazo avec les coordonées du batiment + un tout petit delta histoire d'être sûr qu'un seul batiment nous revienne.

J'ai un gros doute niveau UX. On ne récupère les données que du centre de la carte et non plus de toute la carte affichée. Par ex: ![Capture d'écran montrant le problème](https://forge.aquilenet.fr/attachments/8e67de4e-2bcf-40b0-beb3-dc00d15dc91f) Hors, dans ce cas-là, le reste du village pour lequel il n'y a pas de pins est bien dans la base de données ARCEP. Je pense que ça peut perdre pas mal de monde en faisant penser que seul le centre de la carte affichée est couverte. Je pense qu'il vaudrait mieux afficher toutes les données qu'on peut (ie. Arcep et Axione) pour toute la bounding box de la carte du client. Puis, dans les tooltip, ajouter un bouton pour récupérer les données d'éligibilité Liazo pour un batiment précis à la demande de l'utilisateur. Techniquement, derrière, on ferait une requête à Liazo avec les coordonées du batiment + un tout petit delta histoire d'être sûr qu'un seul batiment nous revienne.
felix.baylac added 1 commit 2022-04-15 10:36:08 +02:00
felix.baylac added 1 commit 2022-05-17 12:15:29 +02:00
The previous message was misleading, adopting youpi's suggestion instead.
felix.baylac force-pushed johan/consolidate-ipe-data from 7c424cee9e to 18187c0007 2022-05-17 12:29:22 +02:00 Compare
johan.le.baut added 1 commit 2022-06-21 14:28:06 +02:00
johan.le.baut added 1 commit 2022-06-21 16:56:02 +02:00
johan.le.baut added 1 commit 2022-10-14 10:56:49 +02:00
johan.le.baut added 1 commit 2022-10-14 10:57:24 +02:00
johan.le.baut force-pushed johan/consolidate-ipe-data from aef1c0bebc to 6822ec4866 2022-10-14 13:19:02 +02:00 Compare
johan.le.baut added 1 commit 2022-10-14 13:48:24 +02:00
johan.le.baut merged commit 1e9d62c03b into master 2022-10-14 13:56:32 +02:00
johan.le.baut deleted branch johan/consolidate-ipe-data 2022-10-14 13:56:32 +02:00
johan.le.baut referenced this pull request from a commit 2022-10-14 13:56:33 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Aquilenet/Axione-IPE-Viewer#5
No description provided.