Skip to content

Preserve fractional means for integer timeseries - #12

Open
FLABDUL wants to merge 2 commits into
jathoms:mainfrom
FLABDUL:fix/timeseries-integer-mean
Open

FLABDUL wants to merge 2 commits into
jathoms:mainfrom
FLABDUL:fix/timeseries-integer-mean

Conversation

@FLABDUL

@FLABDUL FLABDUL commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • preserve fractional means for integer PolarsTimeseries values
  • cover the integer-series result 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 #10

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 #13: 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 — 6dfd78a2 Deployed Sep 10, 2026 by FLABDUL via Build docs #21
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.

Integer PolarsTimeseries mean truncates fractional result

1 participant