Conversation
Replace black with ruff as the code formatter across pyproject.toml, Makefile, and CI workflows. Reformat all files with ruff defaults. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
baogorek
added a commit
that referenced
this pull request
Mar 6, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
MaxGhenis
added a commit
that referenced
this pull request
Mar 6, 2026
* Fix enhanced CPS OOM by using half-sample ExtendedCPS The switch from CPS_2024 (frac=0.5) to CPS_2024_Full (frac=1) in commit 1e8d6e1 doubled the household count to 111k, making the 2913-column calibration loss matrix 2.6 GB and causing OOM on 28 GB machines. - Add ExtendedCPS_2024_Half using CPS_2024 (frac=0.5) with its own H5 - Point EnhancedCPS_2024 at the half-sample input (~56k households) - Free loss_matrix and sim objects earlier in the calibration pipeline - Convert loss matrix to float32 before reweighting The full ExtendedCPS_2024 (111k households) is preserved unchanged. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Widen poverty rate test threshold to 30% and reformat Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Reformat with ruff (CI switched from black to ruff in #577) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix CI checkpoint cache invalidation on code changes Scope checkpoint paths by commit SHA so new commits rebuild from scratch instead of restoring stale H5 files from previous builds. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Cache get_current_commit() and move gc import to top-level - Add @functools.cache to avoid repeated subprocess calls for the same commit SHA during a single build (~10+ calls reduced to 1) - Move inline `import gc` to top of loss.py for consistency Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Max Ghenis <mghenis@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
blackwithruff>=0.9.0in dev dependencies[tool.black]section frompyproject.toml(use ruff defaults, line-length 88)Makefileformat target:black . -l 79->ruff format .ruff format --check .instead of theblack-actionruff formatTest plan
ruff format --check .passes (all 153 files formatted)🤖 Generated with Claude Code