Skip to content

Conversation

@LDSamson
Copy link
Collaborator

@LDSamson LDSamson commented Dec 17, 2025

Closes #261 and #110

@LDSamson LDSamson linked an issue Dec 18, 2025 that may be closed by this pull request
@LDSamson LDSamson marked this pull request as ready for review December 22, 2025 15:28
@LDSamson LDSamson linked an issue Dec 22, 2025 that may be closed by this pull request
Copy link
Collaborator

@aclark02-arcus aclark02-arcus left a comment

Choose a reason for hiding this comment

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

@LDSamson, this looks great! Can't wait to try it out on our internal deployment. 🔥
Just need to fix R CMD Check.

Comment on lines -950 to -961
-5,
-5,
-5,
-5,
-5,
-5,
-5,
-5,
-5,
-5,
-5,
-5,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@aclark02-arcus Of note is that one lab limit per data point was drawn previously. This was not shown because the limits overlapped, see here in this snapshot. This is now corrected, which will help with the speed of the figures as well (and it explains the dramatic removal in code lines, see below 😆 )

Image

@LDSamson LDSamson merged commit 9838569 into dev Jan 5, 2026
5 checks passed
@LDSamson LDSamson deleted the ls_add_data_transformation_selection branch January 5, 2026 07:33
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.

Study data: include limit lab data in table Feature - Choose between data transformations

3 participants