Skip to content

Revise Flood and Coastal Risk Assessment in Banjul, The Gambia#149

Merged
lydiascarf merged 5 commits into
rdl-0.3from
wbgufra-gmbbanjul
Jun 2, 2026
Merged

Revise Flood and Coastal Risk Assessment in Banjul, The Gambia#149
lydiascarf merged 5 commits into
rdl-0.3from
wbgufra-gmbbanjul

Conversation

@bennyistanto
Copy link
Copy Markdown
Member

This PR will remove the old version (existing) of JSON metadata:

With the revised version, which include restructure resources list following the individual download link from DDH.

The dataset described in here https://github.com/GFDRR/rdl-datapipeline/issues/17

This PR will remove the old version (existing) of JSON metadata:
- PR: #96
- JSON: https://github.com/GFDRR/rdl-jkan/blob/rdl-0.3/_datasets/json/rdls_lss-gmb_floodcoastres.json
- JKAN: https://jkan.riskdatalibrary.org/datasets/rdls_lss-gmb_floodcoastres/

With the revised version, which include restructure `resources` list following the individual download link from DDH.

The dataset described in here GFDRR/rdl-datapipeline#17
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 1, 2026

📊 PR Sync Results

📝 Dataset Changes

  • ✅ Added: 0 datasets
  • 📝 Modified: 1 datasets
  • 🗑️ Deleted: 2 datasets
📝 Modified Datasets (1)
  • rdls_hevl-gmb_iucrp_floodcoastres
🗑️ Deleted Datasets (2)
  • rdls_hel-lbr_monroviafloodrisk
  • rdls_lss-gmb_floodcoastres

🧮 Vectors

  • ✅ Vectors generated/updated (175 datasets)

✅ Validation

  • ✅ 1/1 datasets passed validation

🧪 Search Tests

  • ❌ 0/1 tests passed
❌ Failed: "vulnerability flood"

Description: Test adapted from comments on Issue #31

Expected includes:

Dataset Expected Found Rank Score
rdls_vln-jrc_glb_flood 3 0.4786
rdls_lss-gmb_floodcoastres None None
rdls_vln-undrr_glb_floodinfrastructure 23 0.4113
rdls_vln-wb_sfrarr_flood 21 0.4148
rdls_hevl-kentmrwcities_nairobi 27 0.3985
rdls_hevl-kentmrwcities_nakuru 15 0.4268
rdls_lss-tmrwcities_daressalaam 29 0.3821
rdls_hevl-bgdtmrwcities_chattogram 35 0.3761
rdls_hevl-ecutmrwcities_quito 66 0.2947
rdls_hevl-npltmrwcities_rapti 70 0.2813
rdls_hevl-psetmrwcities_nablus 79 0.2578
rdls_lss-tmrwcities_coxsbazar 38 0.3701
rdls_lss-tmrwcities_daressalaam 29 0.3821

Missing includes: rdls_lss-gmb_floodcoastres, rdls_hevl-ecutmrwcities_quito, rdls_hevl-npltmrwcities_rapti, rdls_hevl-psetmrwcities_nablus

Top 5 actual results:

Rank Dataset Score
1 rdls_hzd-wb_sfrarr_floodscenario 0.5040
2 rdls_hzd-fathom_glb_floodoac 0.4961
3 rdls_vln-jrc_glb_flood 0.4786
4 rdls_vln-wb_sfrarr_earthquake 0.4706
5 rdls_hevl-mdg_grma_fluvialfloodrisk 0.4668

@bennyistanto bennyistanto requested a review from lydiascarf April 1, 2026 09:53
Copy link
Copy Markdown
Collaborator

@lydiascarf lydiascarf left a comment

Choose a reason for hiding this comment

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

lgtm! just added a quick fix so that the loss hazard type would render correctly

@lydiascarf lydiascarf merged commit ec5b44e into rdl-0.3 Jun 2, 2026
1 check passed
@lydiascarf lydiascarf deleted the wbgufra-gmbbanjul branch June 2, 2026 11:34
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.

2 participants