Skip to content

split keywords by , in facet, then recalculate - #1067

Open
pvgenuchten wants to merge 1 commit into
geopython:masterfrom
pvgenuchten:soilwise-kw-test
Open

pvgenuchten wants to merge 1 commit into
geopython:masterfrom
pvgenuchten:soilwise-kw-test

Conversation

@pvgenuchten

Copy link
Copy Markdown
Contributor

Overview

This PR introduces a procedure which splits keywords by ',' and recalculates totals after db query
Similar mechanism could be used for other fields which are ,-separated
I see 2 alternatives for this PR:

  • keywords not to be stored as concatenated string, but actual list
  • introduce an in-db procedure to split the keywords-string as part of the facet query

Related Issue / Discussion

resolves #1065

Additional Information

image

Contributions and Licensing

(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
  • I have already previously agreed to the pycsw Contributions and Licensing Guidelines

@pvgenuchten

Copy link
Copy Markdown
Contributor Author

we found an interesting side effect here (potentially related)

if a term contains a space, the aggregated statistics provide a much higher number then when you filter by that term

image
after filtering by 'water holding capacity'
image

@github-actions

Copy link
Copy Markdown

This Pull Request has been inactive for 90 days. In order to manage maintenance burden, it will be automatically closed in 7 days.

@github-actions github-actions Bot added the stale label Sep 21, 2025
@github-actions

github-actions Bot commented Oct 5, 2025

Copy link
Copy Markdown

This Pull Request has been closed due to there being no activity for more than 90 days.

@github-actions github-actions Bot closed this Oct 5, 2025
@pvgenuchten

Copy link
Copy Markdown
Contributor Author

hi team, i wonder your thoughts on this item, this item is quite of interest to us (and others i assume)

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.

[ogcapi-records] keywords facet should split keywords to list

1 participant