johan/consolidate-ipe-data #5

Merged
johan.le.baut merged 2 commits from johan/consolidate-ipe-data into master 2 months ago
Owner
There is no content yet.
johan.le.baut added 11 commits 8 months ago
felix.baylac reviewed 8 months ago
echo "[+] Ingesting IPE data from ${fullIpePath}"
echo ""
cat > "${tmpSql}" <<EOF

CF. #6

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

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

facepalm

*facepalm*
johan.le.baut marked this conversation as resolved
[DB]
path = /path/to/ipe.sqlite
axione_ipe_path = /path/to/ipe.sqlite

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.

Résolu.

Résolu.

Résolu.

Résolu.
felix.baylac marked this conversation as resolved
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")

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' ```

Solved

Solved
johan.le.baut marked this conversation as resolved
# from .cursor import *

Fichier vide, à supprimer.

Fichier vide, à supprimer.

Facepalm, c'est l'init.

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

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
# nex=processed_args['nex'],
# ney=processed_args['ney'],
# )
narrow_coordinates=AreaCoordinates(

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?
Poster
Owner

oui c'était trop large sinon

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

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
const minZoomForRequest = 17;

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
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}`);

Pareil: enlever centerlat et centerlng.

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

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 8 months ago
felix.baylac left a comment
Owner

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 8 months ago
felix.baylac added 1 commit 7 months ago
7c424cee9e
webapp: alter Kosc/Bouygues msg
felix.baylac force-pushed johan/consolidate-ipe-data from 7c424cee9e to 18187c0007 7 months ago
johan.le.baut added 1 commit 6 months ago
johan.le.baut added 1 commit 6 months ago
johan.le.baut added 1 commit 2 months ago
johan.le.baut added 1 commit 2 months ago
johan.le.baut force-pushed johan/consolidate-ipe-data from aef1c0bebc to 6822ec4866 2 months ago
johan.le.baut added 1 commit 2 months ago
johan.le.baut merged commit 1e9d62c03b into master 2 months ago
johan.le.baut deleted branch johan/consolidate-ipe-data 2 months ago

Reviewers

felix.baylac requested changes 8 months ago
The pull request has been merged as 1e9d62c03b.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Aquilenet/Axione-IPE-Viewer#5
Loading…
There is no content yet.