Skip to content

fix(SIDAS): reverse-score item 2 as 10 - control - #54

Merged
joshunrau merged 1 commit into
mainfrom
fix-sidas-reverse-scoring
Oct 29, 2026
Merged

joshunrau merged 1 commit into
mainfrom
fix-sidas-reverse-scoring

Conversation

@joshunrau

Copy link
Copy Markdown
Collaborator

Fixes #51.

totalScore added controlThoughts - 10 instead of reverse-scoring the controllability item as 10 - controlThoughts. That gave a range of -10 to 40 instead of the published 0 to 50, and less control lowered the total.

With the fix, the examples from the issue come out as expected:

Item 1 Item 2 Items 3–5 Total
10 0 10, 10, 10 50
1 0 0, 0, 0 11
5 10 0, 0, 0 5

The PR also bumps internal.edition from 1 to 2. No series instrument references SIDAS. pnpm lint passes.

🤖 Generated with Claude Code

The total score subtracted 10 from the controllability item instead of reversing it, giving a range of -10 to 40 instead of 0 to 50. Bump edition to 2.

Fixes #51

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Form previews

Open the forms changed in this pull request in the Open Data Capture playground:

Generated for f7820b5

@joshunrau
joshunrau merged commit 8a56047 into main Oct 29, 2026
2 checks passed
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.

SIDAS: totalScore reverse-scores item 2 (control) with the wrong sign

1 participant