Allow skipping data logging for a trial with `record_data: false` #3167

jodeleeuw · 2023-10-30T13:50:01Z

This adds record_data as a trial parameter. If set to false then data collection is skipped for that trial.

var trial = {
  type: jsPsychImageKeyboardResponse,
  stimulus: 'imgA.jpg',
  record_data: false
}

This closes #687.

changeset-bot · 2023-10-30T13:50:05Z

Latest commit: 007d862

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
jspsych	Minor

packages/jspsych/src/modules/data/index.ts

added record_data parameter

6f9d01b

jodeleeuw linked an issue Oct 30, 2023 that may be closed by this pull request

Allow skipping data logging for a trial #687

Closed

jodeleeuw requested a review from bjoluc October 30, 2023 13:50

jodeleeuw added this to the 8.0 milestone Oct 30, 2023

bjoluc reviewed Nov 8, 2023

View reviewed changes

packages/jspsych/src/modules/data/index.ts Outdated Show resolved Hide resolved

jodeleeuw and others added 4 commits November 10, 2023 16:44

add test for #3167 (comment)

e3b2acc

return undefined in getResult when record_data is false

33c97c2

Handle undefined results

ab7ad5a

Add a getResult[s]() unit test for the record_data: false parameter

007d862

bjoluc linked an issue Dec 6, 2023 that may be closed by this pull request

Removal of on_data_update event #2736

Open

jodeleeuw merged commit 2898da6 into v8 Dec 8, 2023
2 checks passed

jadeddelta deleted the skip-data-logging branch December 25, 2024 01:54

Provide feedback