Skip to content

Promote mixed numeric timeseries arithmetic to float - #13

Open
FLABDUL wants to merge 2 commits into
jathoms:mainfrom
FLABDUL:fix/timeseries-numeric-promotion
Open

FLABDUL wants to merge 2 commits into
jathoms:mainfrom
FLABDUL:fix/timeseries-numeric-promotion

Conversation

@FLABDUL

@FLABDUL FLABDUL commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • promote mixed integer/float timeseries arithmetic to float when either operand is float
  • cover scalar and timeseries operands on both configured market backends
  • make pull-request CI deterministic by using the locked Ruff version and reserving the protected Pages environment for release deployments

Closes #11

Validation

  • focused timeseries tests — 26 passed
  • full test suite — 262 passed, 3 skipped
  • Ruff check and format check
  • targeted Pyrefly check — 0 errors

Shared CI repair

This PR contains the same CI prerequisite commit as #9 and #12: Ruff is read from uv.lock using the Node 24 action, and GitHub Pages deployment is split from the pull-request docs build. The shared commit only needs to land once; it can be dropped from later PRs after the first is merged.

This branch had an error being deployed

1 failed (outdated) deployment
github-pages — 4a98d3ea Deployed Sep 10, 2026 by FLABDUL via Build docs #22
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.

Mixed numeric PolarsTimeseries arithmetic can truncate floats

1 participant