-
Notifications
You must be signed in to change notification settings - Fork 16
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
[BRMO-352] - Toevoegen van nHr-views #2090
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2090 +/- ##
=========================================
- Coverage 40% 40% -0%
- Complexity 807 945 +138
=========================================
Files 371 371
Lines 18806 18806
Branches 1774 1774
=========================================
- Hits 7488 7487 -1
Misses 10725 10725
- Partials 593 594 +1 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2090 +/- ##
========================================
+ Coverage 38% 38% +1%
Complexity 806 806
========================================
Files 371 371
Lines 18863 18863
Branches 1777 1777
========================================
+ Hits 7153 7155 +2
+ Misses 11138 11136 -2
Partials 572 572 |
@mprins Wat moet er nu nog met dit PR gebeuren? Ik zie een aantal checks die niet completed zijn, maar dit is alleen een 'extra script', dus waar hangt dit nu precies op? Met een link naar de basisprocedure kan ik niks en ik denk dat ik het mag oplossen aangezien geert niet meer aanwezig is. |
CI herstart |
02d56dd
to
5f1b004
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KJLammers voor zover ik na kan gaan staat er niets in de upgrade instructies: https://github.com/B3Partners/brmo/wiki/Upgrade-v3.0.2-naar-v4.0.0 of installatie handleiding.
Tevens worden deze scripts niet opgenomen in de docker image; daarvoor moeten ze aan de volgende lijstjes worden toegevoegd:
Lines 142 to 145 in bc5aa92
<includes> <include>**/209_*.sql</include> <include>**/210_*.sql</include> </includes> Lines 157 to 160 in bc5aa92
<includes> <include>**/202_*.sql</include> <include>**/1000_*.sql</include> </includes> brmo/docker/src/main/docker/Dockerfile
Lines 41 to 50 in bc5aa92
COPY ["bin_unzipped/db/rsgb/datamodel_postgresql.sql", \ "bin_unzipped/db/rsgb/postgresql/209_bag2_rsgb_views.sql", \ "bin_unzipped/db/rsgb/postgresql/210_bag2_brk2.0_mat_views.sql", \ "bin_unzipped/db/brk/brk2.0_postgresql.sql", \ "bin_unzipped/db/brk/brk2.0_postgresql_views.sql", \ "bin_unzipped/db/brk/brk2.0_commentaar.sql", \ "bin_unzipped/db/bag/postgresql/208_bag2_views.sql", \ "bin_unzipped/db/topnl/postgres.sql", \ "bin_unzipped/db/staging/create-brmo-persistence-postgresql.sql", \ "/opt/brmo-data/sqlscripts/"]
Ik heb aanpassingen doorgevoerd op dit PR. |
aanpassing gedaan
Deze views kunnen ervoor zorgen dat de data van het Handelsregister kan worden gebruikt als kaartlagen in een GIS-viewer. Door de tabellen
subject
envestg
met elkaar te koppelen en te koppelen aan het adresseerbaar object, is het mogelijk om de nHr-informatie op de kaart te tonen. Aan de hand van het adresseerbaar object is een administratieve koppeling te realiseren met de BAG. De BRK is gekoppeled met een spatial join.Het is mogelijk om op drie verschillende manieren de kaartlaag te visualiseren aangezien er drie verschillende geometriëen aanwezig zijn. Om die reden zijn er drie materialized views gemaakt, namelijk:
mb_kvk_adres
: koppelt het adresseerbaarobject met de nHr-gegevens (punt-geometrie).mb_kvk_pand
: koppelt via demaaktdeeluitvan
attribuut bovenstaande materialized view zodat een pand-geometrie kan worden getoond (vlak-geometrie).mb_kvk_perceel
: koppeltmb_kvk_adres
met een spatial join (ST_CONTAINS
) zodat een perceel-geometrie en overige BRK-gegevens kan worden getoond (vlak-geometrie).Na het inladen van nHr-data kunnen deze views worden aangemaakt.