Review ask

As a clinical trial statistician or programmer, challenge anything in this benchmark: describe a problem in Comments, or propose a fix with the Edit button on any source file.

  1. Task: Is this a specification error a study team could realistically make, and should the build stop on it rather than continue?
  2. Inputs: Do they trigger the failure for the reason the summary gives?
  3. Expected failure: Is this the right error, raised at the right point, and would it tell the programmer what to fix?

The other tabs beside Comments are background for assessment, not under review.

Summary

Lifecycle: reviewedEditUnder review
What this benchmark means
README.md

Goal: carry the recorded sex (SEX) for each subject from collected demographics, accepting only M, F, and U.

Input: collected demographics with study and subject identifiers and recorded sex (SEX).

Variables:

  • SEX is the subject's recorded sex, copied from collected SEX. A missing value remains missing and passes the final check; any other value must be M, F, or U.

A non-missing code outside the accepted set is rejected after the dataset is completed, so no artifact is accepted.

How to fix

Confirm the code against the study's sex coding convention. Correct X at the governed source when it is a data-entry error. If X is an intentional study code, document that policy and add it to the anchored accepted set on the SEX column:

verifications:
  - matches:
      pattern: '^[MFUX]$'

Expected failure

Edit
the run is rejected; no artifact is accepted
expected/error.yaml
phase
verification
condition
matches_failed
requirement
REQ-0379
expected/error.yaml
phase: verification
condition: matches_failed
spec_paths:
  - columns.SEX.verifications[0].matches
requirement: REQ-0379
context:
  column: SEX
  failure_count: 1
  keys:
    - STUDYID: PILOT7
      USUBJID: P7-952

Inputs

Under review
1 source file

input/dm.csv

Edit
2 rows
input/dm.csv
STUDYIDUSUBJIDSEX
PILOT7M
PILOT7X

Unexpected Output

Under review
2 rows presented to the failing check - not an accepted artifact

expected/adsl.csv

Edit
2 rows
expected/adsl.csv
STUDYIDUSUBJIDSEX
PILOT7M
PILOT7X
Shaded columns are derived— = missing value

Comments and assessment materials

Comments

Signed in with GitHub; saved in repository Discussions

Mapping spec

Generated from the benchmark YAML; do not edit by hand

Dataset NameVariable NameVariable LabelTypeCodelist/Controlled TermsCoreComputational MethodOrigin
ADSLSTUDYIDStudy IdentifiercharacterCopy value from DM.STUDYID.Predecessor
ADSLUSUBJIDUnique Subject IdentifiercharacterCopy value from DM.USUBJID.Predecessor
ADSLSEXSexcharacterCopy value from DM.SEX.Predecessor
VersionDateAuthorDescriptionReviewerSign-off
1.32026-09-29yamaa docs buildShow study codelist names, external dictionaries, and ISO 8601 formats in variable rows; list the document's codelist values and metadata
1.22026-09-29yamaa docs buildClassify ADaM copies from a declared input as Predecessor and recodes as Derived, and SDTM recodes of locally derived columns as Derived
1.12026-09-29yamaa docs buildResolve lookups into the donor record they select, and give every derivation shape a plain-language rule instead of its raw YAML
1.0yamaa docs buildInitial generation from spec.yaml