Skip to content

(feat) LIME2-714: Add reference range for labs based on diff conditions#181

Draft
vasharma05 wants to merge 7 commits intomainfrom
lime2-714
Draft

(feat) LIME2-714: Add reference range for labs based on diff conditions#181
vasharma05 wants to merge 7 commits intomainfrom
lime2-714

Conversation

@vasharma05
Copy link
Collaborator

@vasharma05 vasharma05 commented May 27, 2025

Haemoglobin

image

Hematocrit

image

Lymphocytes

image

Red blood cells

image

Serum glucose

image

Total Billrubin

image

White blood cells

image

C Reactive protein

image

Serum creatinine

image

🐞 Related Issues / Tickets

Link to any existing issues, feature requests, or bugs this PR addresses.

  • Closes #LIME2-714

✅ PR Checklist

👨‍💻 For Author

  • I included the JIRA issue key in all commit messages (e.g., LIME2-123)
  • I wrote or updated tests covering the changes (if applicable)
  • I updated documentation (if applicable)
  • I verified the feature/fix works as expected
  • I attached a video or screenshots showing the feature/fix working on DEV
  • I logged my dev time in JIRA issue
  • I updated the JIRA issue comments, status to "PR Ready"
  • I unassigned the issue so a reviewer can pick it up
  • I mentionned its status update during dev sync call or in Slack
  • My work is ready for PR Review on DEV

🔍 For Reviewer

  • I verified the JIRA ticket key is present in commits
  • I reviewed and confirmed the JIRA issue status is accurate
  • I verified the feature/fix works as expected on DEV or UAT
  • I attached a video or screenshots showing the feature/fix working on DEV or UAT
  • I updated the JIRA issue comments, status to "DEV Ready" or "UAT Ready"
  • I logged my review time in JIRA issue
  • I mentionned its status update during daily dev sync call or on Slack
  • This work is ready for UAT and PRODUCTION

PR Summary by Typo

Overview:
This PR adds reference ranges for various lab tests based on different patient conditions (age, gender, pregnancy). A new CSV file conceptreferenceranges.csv is introduced containing these ranges. Additionally, the PR updates the referenceApplicationDockerVersion and removes some unused dependencies.

Key Changes:

  • Added a new CSV file defining reference ranges for various lab tests based on patient criteria like age, gender, and pregnancy status.
  • Updated the referenceApplicationDockerVersion to 3.4.0.
  • Removed unused OpenMRS dependencies from the pom.xml.
  • Modified the build process to exclude specific Ozone configurations and update the docker image version.

Recommendations:
Not deployment ready. The criteria expressions in the CSV need review to ensure they are valid and cover all intended cases. Consider adding validation tests for the CSV data.

🗂️ Work Breakdown

Category Lines Changed
New Work 36 (66.7%)
Churn 15 (27.8%)
Rework 3 (5.6%)
Total Changes 54
To turn off PR summary, please visit Notification settings.

@typo-app
Copy link
Contributor

typo-app bot commented May 27, 2025

Oops! Your Typo free trial has ended. If you'd like to continue receiving code reviews, please ask the admin to check out our special pricing plan.

@vasharma05 vasharma05 marked this pull request as draft May 28, 2025 10:39
@vasharma05 vasharma05 marked this pull request as ready for review May 29, 2025 07:53
@pirupius pirupius changed the title LIME2-714: Add reference range for labs based on diff conditions (feat) LIME2-714: Add reference range for labs based on diff conditions Oct 2, 2025
@vasharma05 vasharma05 marked this pull request as draft October 28, 2025 06:15
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.

1 participant