Skip to content

Filter cagemates to current public housing and guard null locations #747

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

Open
labkey-martyp wants to merge 3 commits into
base: release26.7-SNAPSHOT
Choose a base branch
from
+12 −5
Open

Filter cagemates to current public housing and guard null locations #747

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d4d1e33
Filter cagemates to current public housing and guard null locations
labkey-martyp Aug 23, 2026
0916041
Filter cagemates on demographics QC state
labkey-martyp Aug 24, 2026
cdb203c
Filter cagemates on both sides and drop redundant room grouping
labkey-martyp Aug 24, 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
17 changes: 12 additions & 5 deletions nirc_ehr/resources/queries/study/demographicsCagemates.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,19 @@ SELECT
FROM study.housing h

JOIN study.housing h2
ON (h2.Id.demographics.calculated_status = 'Alive'
AND (h.cage = h2.cage))
ON (h.cage = h2.cage
AND h2.Id.demographics.calculated_status = 'Alive'
AND h2.Id.demographics.qcstate.publicdata = true
AND h2.enddateTimeCoalesced >= now()
AND h2.qcstate.publicdata = true)

WHERE h.enddateTimeCoalesced >= now()
GROUP BY h.id, h.room, h.cage
-- cage holds the ehr_lookups.cage location key, so a null means this row's location never resolved; such a row gets no cagemates rather than sharing one group with every other unresolved row
WHERE h.cage IS NOT NULL
AND h.enddateTimeCoalesced >= now()

@ankurjuneja ankurjuneja Aug 25, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is there a need to look at the startDate? for a housing record whose date is in the future satisfies it and is treated as current cage mates?

AND h.qcstate.publicdata = true
GROUP BY h.id, h.cage

) t ON (t.id = d.id)

WHERE d.calculated_status = 'Alive'
WHERE d.calculated_status = 'Alive'
AND d.qcstate.publicdata = true