Skip to content

Conversation

@csebranek
Copy link
Collaborator

@csebranek csebranek commented Dec 4, 2024

Rationale

SPI requested a column in ticket. Also added a new blood draw plot and updated queries to calculate current blood amounts.

Related Pull Requests

Changes

@csebranek csebranek requested a review from aschmidt34 December 4, 2024 16:44
Copy link
Collaborator

@aschmidt34 aschmidt34 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@csebranek csebranek merged commit fb3615b into release24.3-SNAPSHOT Dec 4, 2024
2 checks passed
@csebranek csebranek deleted the 24.3_fb_blood_avail_graph branch December 4, 2024 22:02
csebranek added a commit that referenced this pull request Feb 4, 2025
* Add blood graph along with queries to support it, as well as new column for blood avail including thirty days in the future

* exclude today for next 30 days calculation of blood, fix blood summary calc, adjust labels

* use next 30 days since it does not include today

* use ehr version

* remove wnprc blood graph to use ehr instead

* Consolidate blood draw reports into currentBloodDraws.sql, adjust references

* ignore project type

* group up the blood draws if they are done at the same time

* Revert back to original AvailBlood calculation, add new calc for future draws in the new current blood report, fix ehr blood store so all records in transaction are used

* update notification to revert to old blood avail name, use species as lookups for blood queries

* restore demographicsBloodSummary query file

* update test comments

* restore old blood draw query

---------

Co-authored-by: Marty Pradere <martyp@labkey.com>
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.

4 participants