Skip to content

Shorten datasets description on RDL Catalog#155

Merged
lydiascarf merged 2 commits into
rdl-0.3from
shorter-description
Jun 2, 2026
Merged

Shorten datasets description on RDL Catalog#155
lydiascarf merged 2 commits into
rdl-0.3from
shorter-description

Conversation

@bennyistanto
Copy link
Copy Markdown
Member

@bennyistanto bennyistanto commented Apr 13, 2026

resolves #154

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 13, 2026

📊 PR Sync Results

📝 Dataset Changes

  • ✅ Added: 0 datasets
  • 📝 Modified: 0 datasets
  • 🗑️ Deleted: 1 datasets
🗑️ Deleted Datasets (1)
  • rdls_hel-lbr_monroviafloodrisk

🧮 Vectors

  • ✅ Vectors generated/updated (175 datasets)

✅ Validation

  • ✅ 0/0 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 16 0.4246
rdls_vln-undrr_glb_floodinfrastructure 24 0.4113
rdls_vln-wb_sfrarr_flood 22 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_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 13, 2026 23:33
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 cleaned up some of the interactive logic to use a more idiomatic Alpine.js pattern for consistency

@lydiascarf lydiascarf merged commit 790e433 into rdl-0.3 Jun 2, 2026
1 check passed
@lydiascarf lydiascarf deleted the shorter-description 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.

Suggest to shorten datasets description on RDL catalog

2 participants