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

Labels Milestones

johan/consolidate-ipe-data #5

Merged
johan.le.baut merged 2 commits from johan/consolidate-ipe-data into master 6 months ago
Conversation 1 Commits 2 Files Changed 4
johan.le.baut commented 12 months ago
Owner
There is no content yet.
felix.baylac reviewed 12 months ago
data-ingest/ingest_arcep
@ -0,0 +20,4 @@
echo "[+] Ingesting IPE data from ${fullIpePath}"
echo ""
cat > "${tmpSql}" <<EOF
felix.baylac commented 12 months ago
Owner

CF. #6

CF. https://forge.aquilenet.fr/Aquilenet/Axione-IPE-Viewer/issues/6
felix.baylac commented 12 months ago
Owner

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 12 months ago
Owner

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 12 months ago
Owner

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 12 months ago
Owner

Résolu.

Résolu.
felix.baylac commented 12 months ago
Owner

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 12 months ago
Owner

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 12 months ago
Owner

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 12 months ago
Owner

Fichier vide, à supprimer.

Fichier vide, à supprimer.
felix.baylac commented 12 months ago
Owner

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 12 months ago
Owner

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 12 months ago
Owner

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 6 months ago
Poster
Owner

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 12 months ago
Owner

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 12 months ago
Owner

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 12 months ago
Owner

Pareil: enlever centerlat et centerlng.

Pareil: enlever centerlat et centerlng.
johan.le.baut marked this conversation as resolved
felix.baylac commented 12 months ago
Owner

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 12 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.
johan.le.baut merged commit 1e9d62c03b into master 6 months ago
johan.le.baut deleted branch johan/consolidate-ipe-data 6 months ago
johan.le.baut referenced this issue from a commit 6 months ago
johan/consolidate-ipe-data (#5)

Reviewers

felix.baylac requested changes 12 months ago
The pull request has been merged as 1e9d62c03b.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b johan/consolidate-ipe-data master
git pull origin johan/consolidate-ipe-data

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff johan/consolidate-ipe-data
git push origin master
Sign in to join this conversation.
Reviewers
Request review
No reviewers
felix.baylac
Labels
Apply labels
Clear labels   
Acquisition de données
   
Webapp
No Label
Acquisition de données
Webapp
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
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
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'johan/consolidate-ipe-data'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes