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

[BRMO-352] - Toevoegen van nHr-views #2090

Merged
merged 11 commits into from
Nov 14, 2024
Merged

[BRMO-352] - Toevoegen van nHr-views #2090

merged 11 commits into from
Nov 14, 2024

Conversation

geertoff
Copy link
Contributor

Deze views kunnen ervoor zorgen dat de data van het Handelsregister kan worden gebruikt als kaartlagen in een GIS-viewer. Door de tabellen subject en vestg 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:

  1. mb_kvk_adres: koppelt het adresseerbaarobject met de nHr-gegevens (punt-geometrie).
  2. mb_kvk_pand: koppelt via de maaktdeeluitvan attribuut bovenstaande materialized view zodat een pand-geometrie kan worden getoond (vlak-geometrie).
  3. mb_kvk_perceel: koppelt mb_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.

Copy link

codecov bot commented Mar 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 40%. Comparing base (e143fd6) to head (b42aea9).

Additional details and impacted files

Impacted file tree graph

@@            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     

see 1 file with indirect coverage changes

@codecov-commenter
Copy link

codecov-commenter commented May 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38%. Comparing base (130c169) to head (8160042).
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           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           

see 2 files with indirect coverage changes

@mprins
Copy link
Member

mprins commented Jul 4, 2024

@KJLammers
Copy link
Contributor

KJLammers commented Aug 23, 2024

@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.

@mprins
Copy link
Member

mprins commented Aug 24, 2024

CI herstart

@mprins mprins force-pushed the BRMO-352 branch 2 times, most recently from 02d56dd to 5f1b004 Compare October 16, 2024 09:24
Copy link
Member

@mprins mprins left a 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:

  • brmo/brmo-dist/assembly.xml

    Lines 142 to 145 in bc5aa92

    <includes>
    <include>**/209_*.sql</include>
    <include>**/210_*.sql</include>
    </includes>
  • brmo/brmo-dist/assembly.xml

    Lines 157 to 160 in bc5aa92

    <includes>
    <include>**/202_*.sql</include>
    <include>**/1000_*.sql</include>
    </includes>
  • 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/"]

@mvdstruijk
Copy link
Member

Ik heb aanpassingen doorgevoerd op dit PR.
En de upgrade instructies bijgewerkt: https://github.com/B3Partners/brmo/wiki/Upgrade-v3.0.2-naar-v4.0.0#release

@mvdstruijk mvdstruijk requested a review from mprins November 13, 2024 19:31
@mprins mprins enabled auto-merge (squash) November 14, 2024 09:21
@mprins mprins merged commit f89f504 into master Nov 14, 2024
30 checks passed
@mprins mprins deleted the BRMO-352 branch November 14, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants