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: build a subject-level result carrying the enrollment site (SITEID) and the recorded sex (SEX) for each subject.

Input: subject listing records holding the enrollment site (SITEID) and the recorded sex (SEX). One record carries one more field than the header names, and nothing says what the surplus value holds.

Variables:

  • SITEID would be the site the subject enrolled at, taken from the collected listing.
  • SEX would be the recorded sex, taken from the collected listing.

Reading the longer record would mean either dropping a collected value or shifting every value after it into the wrong field, so the run is rejected while reading the listing and no artifact is accepted.

How to fix

Export one field for each named field in every record. Where the extra value is itself collected, name it in the header and give every record the field, left empty where nothing was collected:

STUDYID,USUBJID,SITEID,SEX,SITENM
CTX,CTX-01,701,F,
CTX,CTX-02,702,M,Royal Infirmary

A bare separator inside a value produces the same surplus, so a value holding a separator is quoted rather than left bare.

Expected failure

Edit
the run is rejected; no artifact is accepted
expected/error.yaml
phase
ingest
condition
source_record_width
requirement
REQ-0851
expected/error.yaml
phase: ingest
condition: source_record_width
spec_paths:
  - input.DM.path
requirement: REQ-0851
context:
  dataset: DM
  path: input/dm.csv
  record: 3
  field: 5

Inputs

Under review
1 source file

input/dm.csv

Edit
raw CSV

Showing raw CSV because the source is not a valid rectangular table.

STUDYID,USUBJID,SITEID,SEX
CTX,CTX-01,701,F
CTX,CTX-02,702,M,Royal Infirmary

Unexpected Output

Under review
No artifact is produced

No fixture files in this directory.

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
ADSLSITEIDStudy Site IdentifiercharacterCopy value from DM.SITEID.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