Aquilenet/Axione-IPE-Viewer
7
1
Fork 0
Code Issues 5 Pull requests Projects Releases Wiki Activity

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
Conversation 1 Commits 2 Files changed 4 +52 -41
johan.le.baut commented 2022-04-13 14:37:50 +02:00
Owner
Copy link
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
data-ingest/ingest_arcep
@ -0,0 +20,4 @@
echo "[+] Ingesting IPE data from ${fullIpePath}"
echo ""
cat > "${tmpSql}" <<EOF
felix.baylac commented 2022-04-13 14:47:11 +02:00
Member
Copy link

CF. #6

CF. https://forge.aquilenet.fr/Aquilenet/Axione-IPE-Viewer/issues/6
felix.baylac commented 2022-04-13 15:12:51 +02:00
Member
Copy link

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
shell.nix
@ -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
felix.baylac commented 2022-04-13 14:49:34 +02:00
Member
Copy link

facepalm

*facepalm*
johan.le.baut marked this conversation as resolved
webapp/config.ini.sample Outdated
@ -1,2 +1,5 @@
[DB]
path = /path/to/ipe.sqlite
axione_ipe_path = /path/to/ipe.sqlite
felix.baylac commented 2022-04-13 14:50:32 +02:00
Member
Copy link

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.
felix.baylac commented 2022-04-15 10:36:54 +02:00
Member
Copy link

Résolu.

Résolu.
felix.baylac commented 2022-04-15 10:37:06 +02:00
Member
Copy link

Résolu.

Résolu.
felix.baylac marked this conversation as resolved
webapp/ipe_fetcher/liazo.py
@ -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")
felix.baylac commented 2022-04-13 14:57:05 +02:00
Member
Copy link

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' ```
felix.baylac commented 2022-04-15 10:39:07 +02:00
Member
Copy link

Solved

Solved
johan.le.baut marked this conversation as resolved
webapp/ipe_fetcher/sqlite_connector/__init__.py
@ -0,0 +1 @@
# from .cursor import *
felix.baylac commented 2022-04-13 15:09:26 +02:00
Member
Copy link

Fichier vide, à supprimer.

Fichier vide, à supprimer.
felix.baylac commented 2022-04-15 10:39:43 +02:00
Member
Copy link

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"]:
felix.baylac commented 2022-04-13 15:04:18 +02:00
Member
Copy link

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(
felix.baylac commented 2022-04-13 15:06:24 +02:00
Member
Copy link

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?
johan.le.baut commented 2022-10-14 13:30:56 +02:00
Author
Owner
Copy link

oui c'était trop large sinon

oui c'était trop large sinon
johan.le.baut marked this conversation as resolved
webapp/templates/app.js Outdated
@ -2,3 +72,3 @@
let markers = [];
function initMap(btn) {
// Init map position/zoom. Potentially using what's in the URL search string.
felix.baylac commented 2022-04-13 15:08:08 +02:00
Member
Copy link

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
webapp/templates/app.js
@ -1,118 +1,236 @@
const minZoomForRequest = 17;
felix.baylac commented 2022-04-13 15:11:39 +02:00
Member
Copy link

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
webapp/templates/app.js Outdated
@ -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}`);
felix.baylac commented 2022-04-13 15:08:45 +02:00
Member
Copy link

Pareil: enlever centerlat et centerlng.

Pareil: enlever centerlat et centerlng.
johan.le.baut marked this conversation as resolved
felix.baylac commented 2022-04-13 15:12:12 +02:00
Member
Copy link

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
Copy link

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
johan/consolidate-ipe-data (#5)
Sign in to join this conversation.
Reviewers
No reviewers
felix.baylac
Labels
Clear labels   
Acquisition de données

   
Webapp
No labels
Acquisition de données
Webapp
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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.
Delete branch "johan/consolidate-ipe-data"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?