Review ask

As a clinical trial statistician or programmer, answer two short questions in Your review: the task and the inputs. Agreeing with both approves this benchmark; disagreeing with either needs a comment. Propose a fix with the Edit button on any source file, or add it to your comments.

We suggest spending no more than 5 minutes on a review. The other tabs beside Your review are background for assessment, not under review.

Summary

Lifecycle: draftEditUnder review
What this benchmark means
README.md

Public entries: Python and R. Passing coverage requires an installed inventory.

Goal: carry the investigator comment (CMNT) for each subject, one record per subject.

Input: collected subject listing holding the investigator comment (COMMENT).

Variables:

  • CMNT would be the comment the investigator recorded about the subject, copied from COMMENT.

One comment closes its quoting and then carries on with more text before the next separator. Whether that text belongs to the comment, starts a new value or is dropped depends on the program that reads the file, so the comment has no single reading. The run is rejected while the listing is read, and no artifact is accepted.

How to fix

Have the program that writes the listing quote the whole comment, so the closing quotation mark comes right before the separator or the end of the record:

STUDYID,USUBJID,COMMENT
CTX,CTX-01,Dose reduced
CTX,CTX-02,"Visit missed by site"
CTX,CTX-03,

If the quotation marks belong to the comment itself, double each one inside the quoting, as in """Visit missed"" by site".

Expected failure

Edit
the run is rejected; no artifact is accepted
expected/error.yaml
phase
ingest
condition
source_text_after_quote
rule
CSV-16
expected/error.yaml
phase: ingest
condition: source_text_after_quote
spec_paths:
  - input.DM.path
rule: CSV-16
context:
  dataset: DM
  path: input/dm.csv
  record: 3
  field: 3

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,COMMENT
CTX,CTX-01,Dose reduced
CTX,CTX-02,"Visit missed" by site
CTX,CTX-03,

Unexpected Output

Under review
No artifact is produced

No fixture files in this directory.

Shaded columns are derived— = missing value

Human review and assessment

Your review

2 questions and your comments · View benchmark 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
ADSLCMNTInvestigator CommentcharacterCopy value from DM.COMMENT.Predecessor
VersionDateAuthorDescriptionReviewerSign-off
1.72026-10-20yamaa docs buildRead a specification in the 1.0 surface as its 0.5 counterpart; state that missing values stay missing where a mapping declares no missing handler, and leave out a declared missing answer that restates a handler's default
1.62026-10-17yamaa docs buildDescribe collected date parts written as ISO 8601 date text and times of day counted in seconds since midnight
1.52026-10-17yamaa docs buildName collected precision in words instead of letter codes; state imputation fallbacks, minimum source precision, bounds ignored outside the collected period, and invalid text with no handler as an error; describe date and datetime shifts, imputation flag codes, and partial text giving no date part
1.42026-10-15yamaa docs buildState datetime precision codes through the hour and minute, and complete only the time parts a collected datetime lacks
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