johan/add-api #7

Merged
johan.le.baut merged 17 commits from johan/add-api into master 3 months ago
Owner

Ajout d'une API pour FDN afin d'obtenir le statut d'éligibilité Axione pour une ref. d'immeuble donnée

Modifs dans le script d'ingestion:

  • Arrêter de recopier les headers de chaque .csv dans la table
  • Créer une table parallèle avec IdentifiantImmeuble en index pour req rapide pour l'api
Ajout d'une API pour FDN afin d'obtenir le statut d'éligibilité Axione pour une ref. d'immeuble donnée Modifs dans le script d'ingestion: - Arrêter de recopier les headers de chaque .csv dans la table - Créer une table parallèle avec IdentifiantImmeuble en index pour req rapide pour l'api
johan.le.baut added 2 commits 7 months ago
johan.le.baut requested review from alexandre.borrut 7 months ago
johan.le.baut requested review from samuel.thibault 7 months ago
johan.le.baut requested review from felix.baylac 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 6 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 6 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 5 months ago
johan.le.baut added 1 commit 5 months ago
johan.le.baut added 1 commit 5 months ago
johan.le.baut added 1 commit 5 months ago
johan.le.baut added 1 commit 5 months ago
johan.le.baut added 1 commit 5 months ago
felix.baylac approved these changes 4 months ago
@ -45,4 +45,3 @@
rc=0
echo "Create out dir ${dir_out} if not exist"
mkdir -p "${dir_out}"
Owner

Une raison particulière d'enlever les quotes sur tous ces chemins?

Une raison particulière d'enlever les quotes sur tous ces chemins?
johan.le.baut marked this conversation as resolved
@ -0,0 +2,4 @@
import math
from ipe_fetcher import AreaCoordinates
def check_coordinates_area(coordinates: AreaCoordinates, max_area) -> AreaCoordinates:
Owner

We changed the function behaviour. Let's align the name with the new function.

Something like:

def adapt_coordinates_to_max_area
We changed the function behaviour. Let's align the name with the new function. Something like: ``` def adapt_coordinates_to_max_area ```
johan.le.baut marked this conversation as resolved
johan.le.baut added 1 commit 3 months ago
johan.le.baut merged commit 0fb3de88e1 into master 3 months ago
johan.le.baut deleted branch johan/add-api 3 months ago

Reviewers

alexandre.borrut was requested for review 7 months ago
samuel.thibault was requested for review 7 months ago
felix.baylac approved these changes 4 months ago
The pull request has been merged as 0fb3de88e1.
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/add-api master
git pull origin johan/add-api

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff johan/add-api
git push origin master
Sign in to join this conversation.
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#7
Loading…
There is no content yet.