Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ref_geo point & cor area linear #12

Merged
merged 2 commits into from
Sep 14, 2023
Merged

ref_geo point & cor area linear #12

merged 2 commits into from
Sep 14, 2023

Conversation

joelclems
Copy link
Contributor

ajout d'un référentiel pour les points

@camillemonchicourt
Copy link
Member

OK, en effet pourquoi pas avoir une couche de point, comme on a déjà les zonages polygones et les linéaires.
Ça pourrait servir pour un référentiel de lieux-dits ou autre.

Par contre, je ne comprends pas l'usage de ref_geo.cor_areas et ref_geo.cor_linear_area, et comment ces tables seront remplies ? Par triggers ?

@joelclems
Copy link
Contributor Author

Je pensait ne pas mettre de trigger pour les remplir dans un premier temps

  • pour cor_areas (je n'ai pas trouvé de requete efficace pour gérer les communes, département, régions dans un temps raisonnable, (le st_intersect ne suffit pas, il y a des effets de bord)
  • pour cor_linear ça ce passe mieux, un st_intersects suffit

Pour l'utilité
cor_linear permet de voir rapidement quel lineaire (route, etc..) est dans un departement, et inversement

pour cor_areas cela pourrait permettre de faire le liens entre une region et ses departement

@camillemonchicourt
Copy link
Member

C'est un peu dangereux d'utiliser ces nouvelles tables de correspondance mais de ne pas les remplir automatiquement.
Si on modifie ou ajoute des zonages ou autre, ces tables ne seront pas forcément à jour.
Mais ça peut être lourd encore ajouter des triggers qui lancent des gros calculs à chaque fois qu'on remplit ou modifie le ref_geo...

A minima, indiquer les requêtes à utiliser pour les remplir ?

Pour les intersections des polygones, tu as du rencontrer le même soucis que dans PnX-SI/gn_module_ZH#31
A cause des zonages administratifs qui ont des frontières communes.

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Patch coverage is 85.93% of modified lines.

❗ Current head 742c37d differs from pull request most recent head f997eda. Consider uploading reports for the commit f997eda to get more accurate results

Files Changed Coverage
.../migrations/versions/dea1645de8c0_ref_geo_point.py 68.75%
...igrations/versions/795f6ea8ec45_cor_linear_area.py 71.42%
src/ref_geo/models.py 100.00%

📢 Thoughts on this report? Let us know!.

@joelclems joelclems force-pushed the feat/ref_geo_point branch 2 times, most recently from ec8d896 to cec8534 Compare August 4, 2023 13:36
def upgrade():
stmt = text(importlib.resources.read_text("ref_geo.migrations.data", "ref_geo_cor.sql"))
op.get_bind().execute(stmt)
pass
Copy link
Member

@TheoLechemia TheoLechemia Sep 14, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Petit oubli, à enlever

src/ref_geo/models.py Outdated Show resolved Hide resolved
deferred=True,
)


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il n'y a pas de routes associées à ces modèles ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non, cela reste à faire

@TheoLechemia TheoLechemia changed the base branch from master to develop September 14, 2023 14:44
@TheoLechemia TheoLechemia merged commit 682aa1b into develop Sep 14, 2023
5 checks passed
@camillemonchicourt camillemonchicourt deleted the feat/ref_geo_point branch September 17, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants