Process Capability: Adjust parameter estimation precision settings

[JTPetter]

• fixes [Bug]: Capability - Non normal data - Parameter estimation jasp-issues#4142
• Adjusted precision settings, where it leads to results closer to other software
• Adjusted unit tests (decimal changes) and created new unit tests based on [Bug]: Capability - Non normal data - Parameter estimation jasp-issues#4142

[Copilot]

Pull request overview

Adjusts non-normal parameter estimation behavior in the Process Capability Studies analysis to better match results from other software (per jasp-issues#4142), with corresponding test and snapshot updates.

Changes:

• Refactors/standardizes distribution parameter estimation usage (notably for Weibull/lognormal) and related probability-table calculations.
• Fixes operator-precedence issues in process capability plot x-axis limit expansion for historical parameters.
• Updates/adds unit tests, snapshots, and introduces an additional real-world dataset for verification.

Reviewed changes

Copilot reviewed 4 out of 30 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
R/processCapabilityStudies.R	Fixes precedence in historical-limit logic; refactors probability-table parameter columns and AD/p-value computation flow.
R/commonQualityControl.R	Aligns distribution fitting behavior by removing custom fitdist control tolerances in shared QC helpers.
tests/testthat/test-processCapabilityStudies.R	Updates numeric expectations; adds additional verification coverage for Weibull using a new dataset; updates plot snapshots.
tests/testthat/datasets/processCapabilityStudy/realDataExample2.csv	Adds a new dataset to reproduce/verify the reported discrepancy scenario.
tests/testthat/_snaps/processCapabilityStudies/*.svg	Updates and adds vdiffr snapshots affected by the estimation changes and new tests.
renv.lock	Modifies the recorded metadata for renv in the lockfile.

[Copilot]

In .qcProbabilityTable(), the AD column is declared as type integer, but the code computes and stores non-integer Anderson–Darling statistics (rounded to .numDecimals). This will likely display incorrectly in the UI (e.g., 0.43 shown as 0). Change the column type to number (and optionally add a numeric format) to match the actual values.

Suggested change

	table$addColumnInfo(name = "ad", title = gettext("AD"), type = "integer")
	table$addColumnInfo(name = "ad", title = gettext("AD"), type = "number")

[Copilot]

The p-value approximation branches mix adjusted and unadjusted AD statistics: the first threshold compares adStatistic to 0.6, while subsequent branches compare adStatisticAdjusted. Since the approximation formulas use the adjusted statistic, the thresholding should be based on the adjusted value consistently (otherwise you can select the wrong branch for small/medium sample sizes).

[codecov-commenter]

Codecov Report

❌ Patch coverage is 94.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.31%. Comparing base (06c481a) to head (2848874).

Files with missing lines	Patch %	Lines
R/processCapabilityStudies.R	94.00%	3 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #462   +/-   ##
=======================================
  Coverage   84.30%   84.31%           
=======================================
  Files          18       18           
  Lines       10388    10389    +1     
=======================================
+ Hits         8758     8759    +1     
  Misses       1630     1630           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.