Skip to content

fix: raise cache memory warning threshold to 16 GiB - #550

Draft
anth-volk wants to merge 1 commit into
mainfrom
fix/memory-warning-16gb
Draft

anth-volk wants to merge 1 commit into
mainfrom
fix/memory-warning-16gb

Conversation

@anth-volk

Copy link
Copy Markdown
Contributor

Fixes #549

Summary

Raise the first simulation cache memory warning from 8 GiB to 16 GiB while retaining the 32 GiB warning.

Changes

  • Use 16 GiB and 32 GiB warning thresholds.
  • Report GiB units consistently in the warning message and internal name.
  • Cover values below 16 GiB, one warning per threshold, and warning-state reset after cache clearing.

Testing

  • 9 passed in tests/test_cache.py
  • Ruff format and lint passed.

This branch starts from PolicyEngine.py 6.2.0 on current main. It does not create a 5.2.x maintenance release or edit the package version manually; the release workflow will assign the next version after merge.

This branch has not been deployed

No deployments
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.

Raise the simulation cache memory warning to 16 GiB

1 participant