Skip to content

484 - #1820

Merged
SFJohnson24 merged 2 commits into
mainfrom
fix
Aug 11, 2026
Merged

484#1820
SFJohnson24 merged 2 commits into
mainfrom
fix

Conversation

@SFJohnson24

@SFJohnson24 SFJohnson24 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

small PR for CORE-000484; it is dropping blank USUBJID but blank USUBJID in RELREC are allowed from my understanding.

for the negative/01 open rules test data it drops all the data where usubjid is blank and only reports the last 4 rows. It should report all the rows for the test data as the RELID per USUBJID are unique

@pendingintent pendingintent left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

When validating against CORE conformance rules for a specific implementation, a rule author should double-check if the rule flags null USUBJID in RELREC as an error by default — some validation engines default to flagging it unless explicitly configured to allow the trial-design-relationship case.

The engine should allow null USUBJID so any rule should be authored considering which domains are being validated.

@SFJohnson24
SFJohnson24 merged commit 5046d21 into main Aug 11, 2026
13 checks passed
@SFJohnson24
SFJohnson24 deleted the fix branch August 11, 2026 12:41
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.

2 participants