refactor(ReforceXY): cleanup reward space analysis additive enablement handling

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(pbrs): canonical mode disables additives; migrate prev_potential API

perf(qav3): adjust decline_quantile to 0.75

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): remove unneeded float conversion in vw_path_length calculation

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): cleanup numerical stability checks

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): handle edge cases in zero_phase_filter, smooth_extrema, soft_extremum

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): computation of volume-weighted ER

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(tests): Standardize constants, improve documentation and add docstrings (#22)

* Refactor: Standardize constant access patterns in test suite

Replace class-based constant access (self.TEST_*, self.TOL_*, etc.) with
direct imports from tests/constants module. This improves code clarity and
follows the DRY principle by using the centralized constants module as the
single source of truth.

Changes:
- Replace self.TEST_* with PARAMS.* across all test files
- Replace self.TOL_* with TOLERANCE.* for all tolerance values
- Replace self.SEED* with SEEDS.* for test seeds
- Replace self.CONTINUITY_* with CONTINUITY.*
- Replace self.EXIT_FACTOR_* with EXIT_FACTOR.*
- Replace self.BH_FP_RATE_THRESHOLD with STATISTICAL.BH_FP_RATE_THRESHOLD
- Add/update imports in 12 test files to include required constants
- Fix PNL_BASE_STD -> PNL_STD (correct constant name)

Files modified:
- tests/test_base.py: Updated imports and all constant references
- tests/api/test_api_helpers.py: Added PARAMS, SEEDS, TOLERANCE imports
- tests/cli/test_cli_params_and_csv.py: Added SEEDS import
- tests/components/test_*.py: Updated to use direct imports
- tests/integration/test_*.py: Updated constant access patterns
- tests/pbrs/test_pbrs.py: Comprehensive constant migration
- tests/robustness/test_robustness.py: Updated all robustness tests
- tests/statistics/test_statistics.py: Added required constants

All tests verified passing after migration.

* docs(tests): Add tolerance justification guide and docstring template

Phase 1.2 & 1.3: Comprehensive documentation for test suite best practices

Added:
1. TOLERANCE_GUIDE.md - Comprehensive guide for tolerance selection
   - Decision matrix for choosing appropriate tolerances
   - Detailed rationale for each tolerance constant
   - Usage examples and anti-patterns
   - Mathematical justification for tolerance magnitudes

2. .docstring_template.md - Standardized test docstring format
   - Multiple template levels (minimal, standard, complex)
   - Section-by-section guidelines
   - Common patterns (property-based, regression, integration)
   - Checklist for new tests

3. Tolerance justification comments in test files
   - test_robustness.py: Added 11 inline comments explaining tolerance choices
   - test_pbrs.py: Enhanced docstring with tolerance rationale
   - Focused on IDENTITY_RELAXED and NUMERIC_GUARD usages

4. Fixed test_statistics.py imports
   - Moved constants import to proper location (top of file)
   - Removed duplicate import that was incorrectly placed mid-function

Benefits:
- New contributors can quickly understand tolerance choices
- Reduces test flakiness by documenting error accumulation models
- Establishes clear standards for test documentation
- Makes tolerance selection systematic rather than arbitrary

See also: tests/constants.py for tolerance constant definitions

* docs(tests): Enhance README with constant management and tolerance guidelines

- Add Constants & Configuration section explaining centralized constants.py
- Add Tolerance Selection section with quick reference table
- Add Test Documentation section referencing .docstring_template.md
- Add Maintenance Guidelines section covering:
  - Constant management best practices
  - Tolerance documentation requirements
  - Test documentation standards
- Add Additional Resources section linking to new docs:
  - TOLERANCE_GUIDE.md (numerical tolerance selection guide)
  - .docstring_template.md (test documentation template)
  - constants.py (frozen dataclass constants)
  - helpers/assertions.py (custom assertion functions)
  - test_base.py (base class utilities)

Completes Phase 1 documentation by integrating the tolerance guide and
docstring template into the authoritative test suite README.

* fix(tests): Correct constant reference in bootstrap reproducibility test

Fix incorrect constant migration: SCENARIOS.BOOTSTRAP_ITERATIONS does not
exist. Should use STATISTICAL.BOOTSTRAP_DEFAULT_ITERATIONS instead.

The migration script incorrectly mapped self.BOOTSTRAP_DEFAULT_ITERATIONS
to SCENARIOS.BOOTSTRAP_ITERATIONS when it should have been mapped to
STATISTICAL.BOOTSTRAP_DEFAULT_ITERATIONS (from StatisticalConfig dataclass).

Fixes test_stats_hypothesis_seed_reproducibility in test_statistics.py.

* docs(tests): Clarify TOLERANCE_GUIDE.md as authoritative source

Make it explicit that TOLERANCE_GUIDE.md is the single authoritative source
for tolerance documentation, with README.md providing only a quick reference.

This follows the 'No duplication' principle from copilot-instructions.md:
maintain single authoritative documentation source; reference other sources
rather than copying.

* refactor(tests): Complete constant migration - remove all self.* references

Run migration script to eliminate remaining 28 self.* constant references
across 4 test files:
- test_api_helpers.py: self.SEED → SEEDS.SMOKE_TEST (fixed mapping)
- test_reward_components.py: self.TEST_RR → PARAMS.RISK_REWARD_RATIO (2×)
- test_reward_calculation.py: self.TEST_RR → PARAMS.RISK_REWARD_RATIO (1×)
- test_robustness.py: self.TEST_RR → PARAMS.RISK_REWARD_RATIO (24×)

The migration script (tests/scripts/migrate_constants.py) automates this
refactoring to maintain consistency. It serves two purposes:
1. Complete initial migration (this commit)
2. Future-proof tool for adding new constants or refactoring patterns

All test classes now use direct imports from tests/constants.py.
Class-level aliases in RewardSpaceTestBase can be removed in a follow-up.

* refactor(tests): Remove migration script and unused aliases

- Delete temporary migration script (tests/scripts/migrate_constants.py)
- Remove unused class constant aliases from test_base.py
- Migrate remaining self.PBRS_* references to PBRS.* in test_pbrs.py
- Fix test_case.TOL_IDENTITY_STRICT reference in test_reward_components.py
- Keep only actively used class constants (DEFAULT_PARAMS, PBRS_TERMINAL_PROB, PBRS_SWEEP_ITER, JS_DISTANCE_UPPER_BOUND)

All 154 tests passing.

* docs(tests): Add 21 docstrings to helpers, robustness and statistics tests

Add comprehensive docstrings following .docstring_template.md format:

- tests/helpers/test_internal_branches.py: 3 docstrings
- tests/helpers/test_utilities.py: 6 docstrings
- tests/robustness/test_branch_coverage.py: 5 docstrings
- tests/statistics/test_feature_analysis_failures.py: 7 docstrings

All 29 tests verified passing.

* docs(tests): Remove TOLERANCE_GUIDE.md

* refactor(tests): Complete test suite standardization

- Consolidate mid-file imports (PEP 8 compliance)
- Extend constants.py with new test values
- Update documentation (README, docstring template)
- Fix marker references
- Add initial docstrings to multiple test files

fix(ReforceXY): PBRS trade duration term should be pnl sign-aware

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): do not weight the first pivot if cannot be computed

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): adjust annotation line offset to stoploss

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): adjust take profit annotation line start date

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): bump versions

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor!(qav3): cleanup extrema selection methods namespace

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: refine README.md
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): refine configuration template

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): bump versions

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): add volume weighted efficiency_ratio as extrema weighting source

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(ReforceXY): compute exit factor properly

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): align variable namespace

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): refine take profit plot annotations

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): draw take profit on chart

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): log boolean protections parameters

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): log model and strategy specific configuration

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): cleanup extrema weighting hybrid strategy logic

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(ReforceXY): factor out reward tunable defaults

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(ReforceXY): factor out reward factor computations

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): bump strategy version to 3.3.183

fix(qav3): plot extrema bars with normalized weights

perf(qav3): make extrema weights computation fully numpy-based

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(deps): lock file maintenance (#21)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

docs: refine README.md formatting

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): better handling of 0 weights in extrema weighting

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): align fallbacks un extrema weights handling

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): geometric_mean computation in hybrid extrema weighting

+ nan-aware computations

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): namespace alignment

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs(qav3): refine configuration template

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: add code comment

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs(qav3): refine extrema weighting docs

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): bump versions

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): add hybrid extrema weighting strategy

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): handle NaN with imputation in extrema weighting computation pipeline

When normalizing extrema weights, NaN values can lead to issues in
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: refine README.md description of extrema selection methods

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): add extrema weighting by swing efficiency ratio and speed

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): properly integrate volume extrema weighting, take 2

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor: add pivot volumes calculation in zigzag function

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): ensure extrema weighting computation properly handle NaN

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: improve tunables description

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): bump versions

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: improve tunables description

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: improve tunables description

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: improve README.md configuration tunables description

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): Add volume-weighted amplitude extrema weighting strategy

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): propose a performant configuration template

It's to show what can be achieved by understanding the various tunables
and their interactions.

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): remove unneeded intermediate variables

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): align robust standardization namespace

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): add MMAD standardization for extrema weighting

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): namespace alignment

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): rename savgol params helper

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(qav3): bump QuickAdapterRegressorV3 and QuickAdapterV3 to 3.7.125 and 3.3.175 respectively

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): add 2 extrema smoothing methods: Savitzky–Golay and Nadaraya-Watson

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): ensure symetry at extrema smoothing

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

perf(qav3): sensibler defaults for reversal confirmation

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs(README.md): fix minmax_range type in extrema_weighting config table

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(deps): lock file maintenance (#20)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

refactor(qav3): decouple extrema weighting standardization and normalization

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

perf(qav3): adjust default max_natr_ratio_percent to 0.05

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

feat(qav3): add configuration section for protections

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(deps): lock file maintenance (#19)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

refactor(qav3): factor out optuna MO metrics list

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor: propagate github token to devcontainers

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix: sensible defaults for gh cli devcontainer feature

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: add gh cli feature to devcontainers

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix: workaround nix feature packages installation issue

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix: packages is a comma separated list

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

perf(qav3): refine default config

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(devcontainers): enable attribute path is nix feature

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(devcontainer): handle properly symlinks in workspace

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(devcontainers): optimize devcontainer configurations

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

perf: add cached properties for extrema parameters validation

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

perf(qav3): refine protections default configuration

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor: rename amplitude_excess -> amplitude_threshold_ratio and update docs

chore(deps): lock file maintenance (#18)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

fix(devcontainer): opencode usage on macOS

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: correct default label_weights

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: add docker-in-docker support to devcontainer

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore(devcontainer): move dev tools installation to container start command

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

perf(qav3): sensbile reversal confirmation defaults

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(ReforceXY): remove unnecessary import

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor: improve type hints

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: refine devcontainer configuration

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): improve type hints and variable declarations

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: add opencode vscode plugin to devcontainers

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: update devcontainer and Dockerfile for ReforceXY and QuickAdapter

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: refine devcontainers configuration

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

chore: add devcontainers configuration

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(reforcexy): use proper types

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor(qav3): reuse existing implementation when possible

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix(qav3): fix prediction extrema partition method

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

docs: refine wording

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>