Skip to content

Fix: Resolve column ambiguity error in combined samples query #28

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
merged 1 commit into from
Oct 31, 2025
Merged

Fix: Resolve column ambiguity error in combined samples query #28

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
23 changes: 10 additions & 13 deletions tutorials/parquet_cesium.qmd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -348,10 +348,9 @@ async function get_samples_at_geo_cord_location_via_sample_event(pid) {
SELECT DISTINCT
s.pid as sample_pid,
s.label as sample_label,
s.description as sample_description,
s.thumbnail_url,
s.alternate_identifiers,
s.name as sample_name,
event.label as event_label,
event.pid as event_pid,
site.label as site_label,
site.pid as site_pid,
'direct_event_location' as location_path
Expand All @@ -373,10 +372,9 @@ async function get_samples_at_geo_cord_location_via_sample_event(pid) {
SELECT DISTINCT
s.pid as sample_pid,
s.label as sample_label,
s.description as sample_description,
s.thumbnail_url,
s.alternate_identifiers,
s.name as sample_name,
event.label as event_label,
event.pid as event_pid,
site.label as site_label,
site.pid as site_pid,
'via_site_location' as location_path
Expand All @@ -393,7 +391,7 @@ async function get_samples_at_geo_cord_location_via_sample_event(pid) {
AND g.otype = 'GeospatialCoordLocation'
AND g.pid = ?

ORDER BY thumbnail_url IS NOT NULL DESC, sample_label
ORDER BY sample_label
`;
const result = await loadData(q, [pid, pid], "loading_combined", "samples_combined");
return result ?? [];
Expand Down Expand Up @@ -697,15 +695,14 @@ ${JSON.stringify(samples_2, null, 2)}

<div id="loading_combined" hidden>Loading combined samples…</div>

This query implements Eric Kansa's `get_samples_at_geo_cord_location_via_sample_event` function, which combines both Path 1 and Path 2 using UNION and returns richer sample metadata including:
This query implements Eric Kansa's `get_samples_at_geo_cord_location_via_sample_event` function, which combines both Path 1 and Path 2 using UNION and returns sample metadata including:

- Sample metadata: `sample_pid`, `sample_label`, `sample_description`
- Visual assets: `thumbnail_url`, `alternate_identifiers`
- Event context: `event_label`
- Site information: `site_label`, `site_pid` (when available)
- Sample metadata: `sample_pid`, `sample_label`, `sample_name`
- Event context: `event_label`, `event_pid`
- Site information: `site_label`, `site_pid` (when available via Path 2)
- Path indicator: `location_path` (direct_event_location or via_site_location)

Results are ordered with samples that have thumbnails first, making it easier to find visually rich records.
Results are ordered alphabetically by sample label.

```{ojs}
//| echo: false
Expand Down