Skip to content

docs: query-spec honesty-fix for live search field set (#168 Direction B) #176

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.

Sign up for GitHub

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

Merged
rdhyee merged 2 commits into from
May 8, 2026
Merged

docs: query-spec honesty-fix for live search field set (#168 Direction B) #176

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6f3ba68
docs: honesty-fix query-spec to match live label+place_name (#168 Dir…
rdhyee May 8, 2026
006d335
explorer: add search-limitation UI hint per query-spec MUST clause (#…
rdhyee May 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion explorer.qmd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,9 +239,13 @@ Circle size = log(sample count). Color = dominant data source.

<!-- Static layout: globe + side panel. Updated via DOM, not OJS reactivity. -->
<div class="search-bar">
<input type="text" id="sampleSearch" placeholder="Search samples - multiple words narrow results (e.g., pottery Cyprus)" />
<input type="text" id="sampleSearch" placeholder="Search samples — multiple words narrow results (e.g., basalt California)" />
<button id="searchBtn">Search</button>
</div>
<div class="search-help" style="font-size: 11px; color: #888; padding: 2px 0 6px 0; line-height: 1.3;">
Searches sample labels and place names only — descriptions are not yet indexed.
<a href="https://github.com/isamplesorg/isamplesorg.github.io/issues/169" target="_blank" rel="noopener noreferrer" style="color: #888; text-decoration: underline;">Tracking issue: substrate FTS will lift this</a>.
</div>
<div id="searchResults" class="search-results"></div>

<div class="view-toolbar">
Expand Down
10 changes: 8 additions & 2 deletions query-spec.qmd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,8 +221,14 @@ Solr `searchText` copy-field target, canonical list):
- `curation_label`, `curation_description`, `curation_location`

Substrates that can't index all 15 fields MUST document which subset
they cover and surface the limitation in UI. (The current web Explorer
covers `label` + `description` + `place_name` only — a known gap.)
they cover and surface the limitation in UI. (As of 2026-05-08, the
current web Explorer covers `label` + `place_name` only against
`samples_map_lite.parquet`, which does not carry `description` —
a known gap that broke `pottery Cyprus` and similar queries in the
#167 baseline. The substrate work in
[#169](https://github.com/isamplesorg/isamplesorg.github.io/issues/169)
will lift this by adding `description` *and* dereferenced concept
labels in the v1 minimum field set.)

Multi-term queries default to **AND** with relevance ranking where the
substrate supports it (Solr, DuckDB FTS). See PR #95 for web-side FTS
Expand Down
Loading