Skip to content

Comments

[jaspLearnStats] Change the default value of delta from 0 to 0.5 in Learn Stats -> E...#146

Closed
sisyphus-jasp wants to merge 1 commit intojasp-stats:masterfrom
sisyphus-jasp:fix-sisyphus-1771797321
Closed

[jaspLearnStats] Change the default value of delta from 0 to 0.5 in Learn Stats -> E...#146
sisyphus-jasp wants to merge 1 commit intojasp-stats:masterfrom
sisyphus-jasp:fix-sisyphus-1771797321

Conversation

@sisyphus-jasp
Copy link
Contributor

Summary

PR Summary: Change Default Delta Value in Learn Stats Effect Sizes

Root Cause

The default value for delta (Cohen's δ) in the Learn Stats → Effect sizes analysis was set to 0. When delta is 0, the two distributions (control and experimental) overlap exactly, making the visualization uninformative and unclear for learning purposes.

What Was Changed

  • File: inst/qml/LSeffectSizes.qml
  • Line: 65
  • Change: Changed defaultValue: 0 to defaultValue: 0.5 for the effectSizeValueDelta option

Why This Fix Makes Sense

Setting delta to 0.5 by default provides a more informative visualization:

  • The two distributions are clearly separated but still show overlap
  • This value (0.5) represents a medium effect size, which is commonly used in teaching
  • Students can better understand the concept of effect size when they can see the distributions actually differ

Caveats / Things to Check

  • This is purely a QML default value change - no R code changes were required
  • The change only affects the default value; users can still set delta to any value they want
  • No tests exist for this module (tests/testthat directory is empty), so no regression testing was possible

Implementation Plan

No plan provided by agent.

Test Results

Baseline (before fix): Could not parse test summary
After fix: No tests exist in this module (tests/testthat/ directory is empty).
The change has been verified manually - defaultValue changed from 0 to 0.5 in inst/qml/LSeffectSizes.qml:65

Agent Stats

Turns: 25 | Cost: $1.54 | Duration: 8.2min

Log files
  • JSONL: /home/openclaw/logs/module-developer-jaspLearnStats_1771797321.claude.jsonl
  • Verbose: /home/openclaw/logs/module-developer-jaspLearnStats_1771797321.verbose.log
  • Orchestrator: /home/openclaw/logs/module-developer-jaspLearnStats_1771797321.log

@boutinb boutinb requested a review from JTPetter February 23, 2026 09:45
@FBartos FBartos closed this Feb 23, 2026
@FBartos FBartos removed the request for review from JTPetter February 23, 2026 14:32
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.

2 participants