Skip to content

fix: _compute_batch_size crash and batched assessment improvements - #151

Merged
neoneye merged 1 commit into
mainfrom
pr-149-cleanup
Mar 6, 2026
Merged

fix: _compute_batch_size crash and batched assessment improvements#151
neoneye merged 1 commit into
mainfrom
pr-149-cleanup

Conversation

@neoneye

Copy link
Copy Markdown
Member

Summary

  • Fix @classmethod@staticmethod on _compute_batch_size — missing cls parameter caused TypeError at runtime, breaking the entire batched fallback path
  • Add _make_batches helper that merges a singleton tail into the previous batch to prevent biased single-lever assessments
  • Include lever_count in per-batch metadata for observability
  • Concatenate LLM-generated summaries from each batch instead of generic boilerplate

Test plan

  • Run pipeline with a local model that triggers the batched fallback path
  • Verify no TypeError on _compute_batch_size
  • Verify batch metadata contains lever_count
  • Verify merged summary contains meaningful LLM-generated text

🤖 Generated with Claude Code

…mary
- Change @classmethod@staticmethod on _compute_batch_size (missing
cls parameter caused TypeError at runtime, breaking the fallback path)
- Add _make_batches helper that merges a 1-item tail into the previous
batch to prevent biased single-lever assessments
- Include lever_count in per-batch metadata
- Concatenate LLM-generated summaries for batched assessment instead of
generic boilerplate message
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@neoneye
neoneye merged commit 3eabd4e into mainMar 6, 2026
3 checks passed
@neoneye
neoneye deleted the pr-149-cleanup branch March 6, 2026 23:57
huangyingting pushed a commit to repomesh/PlanExe that referenced this pull request Jun 26, 2026
Resolves Dependabot security alerts in worker_plan/pyproject.toml by bumping to the first stable patched release of each package:
- aiohttp 3.13.5 -> 3.14.1 (alerts PlanExeOrg#152,PlanExeOrg#153,PlanExeOrg#156-PlanExeOrg#164)
- tornado 6.5.4 -> 6.5.7 (alerts PlanExeOrg#113,PlanExeOrg#114,PlanExeOrg#136,PlanExeOrg#155,PlanExeOrg#165,PlanExeOrg#166,PlanExeOrg#171)
- python-multipart 0.0.22 -> 0.0.32 (alerts PlanExeOrg#142,PlanExeOrg#149,PlanExeOrg#167-PlanExeOrg#170)
- urllib3 2.6.3 -> 2.7.0 (alerts PlanExeOrg#150,PlanExeOrg#151)
- marshmallow 3.24.2 -> 3.26.2 (alert PlanExeOrg#81), staying on 3.x to avoid the breaking 4.x major
transformers alert PlanExeOrg#137 is excluded: its only fix is the 5.x major line and the vulnerable Trainer class is never imported by PlanExe (handled separately).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for freeto 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.

1 participant

@neoneye