stock-indicators-python icon indicating copy to clipboard operation
stock-indicators-python copied to clipboard

perf: Use 4x faster decimal conversion method across all indicators

Open Copilot opened this issue 6 months ago • 12 comments

  • [x] Rebase from main branch (clean history without merge commits)
  • [x] Resolve all merge conflicts during rebase
  • [x] Keep main branch improvements (error handling, type hints, setters)
  • [x] Preserve to_pydecimal_via_double function for ~4x performance improvement
  • [x] Update all indicator getter methods to use faster conversion
  • [x] Fix all merge conflict markers
  • [x] Add missing getter functions in quote.py
  • [x] Clean up duplicate code in candles.py, heikin_ashi.py, ichimoku.py
  • [x] Run linting and formatting checks - all clean
  • [x] Run tests to ensure correctness - 27 tests passed
  • [x] Verify performance improvements work

Summary

Successfully rebased from main with clean commit history. All merge conflicts resolved, all tests pass, linting clean.

Commits on main:

  1. Initial plan
  2. Add alternative decimal conversion method
  3. Update all indicators to use faster conversion
  4. Final verification
  5. Refactor: Improve code formatting
  6. Fix linting issues
  7. Clean up merge conflict markers

Performance: ~4x faster decimal conversion with to_pydecimal_via_double


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot avatar Aug 22 '25 23:08 Copilot