Skip to content

fix: prevent stale RSI cache in StochRSI - #36

Merged
stefan-jansen merged 1 commit into
mainfrom
fix/stochrsi-numba-cache
Aug 11, 2026
Merged

fix: prevent stale RSI cache in StochRSI#36
stefan-jansen merged 1 commit into
mainfrom
fix/stochrsi-numba-cache

Conversation

@stefan-jansen

Copy link
Copy Markdown
Contributor

Summary

  • compute RSI outside the cached StochRSI kernel
  • keep the existing StochRSI function signatures and compiled stochastic calculation
  • ensure an upgraded editable installation cannot retain older RSI behavior inside a transitive Numba cache

Root cause

Numba invalidated the cached RSI function after the missing-observation fix, but the separately cached StochRSI caller still embedded the prior compiled RSI implementation. A single missing close could therefore produce finite zero StochRSI values through the end of a series in a long-lived checkout.

Verification

  • deterministic public compute_features(..., ["stochrsi"]) reproduction now passes
  • focused missing-observation and indicator accuracy tests: 49 passed, 2 expected TA-Lib skips before all extras were installed
  • complete all-extras suite: 3,913 passed, 1 skipped, 57 deselected
  • uv run mkdocs build --strict
  • uv build
  • pre-commit run --all-files

Closes #35

Copilot AI lite review requested due to automatic review settings August 11, 2026 17:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@stefan-jansen
stefan-jansen merged commit f88948e into main Aug 11, 2026
25 checks passed
@stefan-jansen
stefan-jansen deleted the fix/stochrsi-numba-cache branch August 11, 2026 20:09
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.

Prevent StochRSI from reusing stale compiled RSI behavior

2 participants