Remove geojson_4326, add geom_4326

[bouttier]

#6

[codecov]

Codecov Report

Attention: 27 lines in your changes are missing coverage. Please review.

Comparison is base (0e8b830) 82.78% compared to head (9c92bac) 81.02%.

Files	Patch %	Lines
..._geo/migrations/versions/bc2fcc772b46_geom_4326.py	37.50%	15 Missing ⚠️
src/ref_geo/env.py	72.72%	3 Missing ⚠️
...ions/0dfdbfbccd63_ref_geo_french_municipalities.py	0.00%	3 Missing ⚠️
...ersions/3fdaa1805575_ref_geo_french_departments.py	25.00%	3 Missing ⚠️
...ns/versions/d02f4563bebe_ref_geo_french_regions.py	0.00%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #17      +/-   ##
===========================================
- Coverage    82.78%   81.02%   -1.76%     
===========================================
  Files           22       24       +2     
  Lines          819      875      +56     
===========================================
+ Hits           678      709      +31     
- Misses         141      166      +25     

Flag	Coverage Δ
pytest	81.02% <66.25%> (-1.76%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mvergez]

Salut !
Ne serait-ce pas possible de plutôt ajouter un INDEX sur ST_Transform(geom, 4326) ?
Niveau perfo vous pensez que ça le ferait ?
Au moins on pourrait avoir qu'une seule colonne géométrique

[camillemonchicourt]

Salut ! Ne serait-ce pas possible de plutôt ajouter un INDEX sur ST_Transform(geom, 4326) ? Niveau perfo vous pensez que ça le ferait ? Au moins on pourrait avoir qu'une seule colonne géométrique

Oui c'est un peu ce que suggère Fred sur le ticket lié #6 (comment) 🤔

[jacquesfize]

Salut @bouttier, @mvergez

J'ai enlevé SQLAlchemy 1.3 sur la GH action de pytest. Est-ce que vous avez d'autres modifications à faire ?