Skip to content

[codex] Add observation-specific multisession costs#1870

Merged
FlorianPfaff merged 8 commits intomainfrom
codex/observation-specific-multisession-costs
Apr 25, 2026
Merged

[codex] Add observation-specific multisession costs#1870
FlorianPfaff merged 8 commits intomainfrom
codex/observation-specific-multisession-costs

Conversation

@FlorianPfaff
Copy link
Copy Markdown
Owner

@FlorianPfaff FlorianPfaff commented Apr 25, 2026

Summary

Adds a small clean follow-up to PR #1751 that keeps the current main multisession solver intact and ports only the missing observation-specific start/end cost wrapper.

Changes

  • Add solve_multisession_assignment_with_observation_costs as a wrapper around the existing multisession solver.
  • Export the new helper from pyrecest.utils.
  • Add focused tests for uniform-cost equivalence, detection-specific start/end costs, scalar broadcasting, threshold handling, and validation errors.
  • Keep Pylint's duplicate-code checker enabled globally; scope the R0801 suppressions to the new wrapper and test modules where the overlap is intentional.

Validation

  • Prior CI passed on commit cc05267c02ac26a9f5e20820e0b3edf90c867274 before this follow-up.
  • This follow-up removes the global duplicate-code disable from pyproject.toml; GitHub CI should rerun on the latest branch head.
  • I could not run the test suite locally from this read-only Codex workspace.

@FlorianPfaff FlorianPfaff marked this pull request as ready for review April 25, 2026 10:35
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ COPYPASTE jscpd yes no no 12.27s
✅ JSON prettier 2 0 0 0 0.43s
✅ JSON v8r 2 0 0 2.68s
✅ MARKDOWN markdownlint 2 0 0 0 0.69s
✅ MARKDOWN markdown-table-formatter 2 0 0 0 0.22s
✅ PYTHON bandit 395 0 0 6.08s
✅ PYTHON black 395 7 0 0 9.64s
✅ PYTHON flake8 395 0 0 3.47s
✅ PYTHON isort 395 11 0 0 0.77s
✅ PYTHON mypy 395 0 0 5.49s
✅ PYTHON pylint 395 0 0 124.54s
✅ PYTHON ruff 395 11 0 0 0.08s
✅ REPOSITORY checkov yes no no 24.81s
✅ REPOSITORY gitleaks yes no no 7.92s
✅ REPOSITORY git_diff yes no no 0.03s
✅ REPOSITORY secretlint yes no no 7.86s
✅ REPOSITORY syft yes no no 3.22s
✅ REPOSITORY trivy-sbom yes no no 2.33s
✅ REPOSITORY trufflehog yes no no 19.88s
✅ YAML prettier 4 0 0 0 0.48s
✅ YAML v8r 4 0 0 5.12s
✅ YAML yamllint 4 0 0 0.45s

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@9.4.0 --custom-flavor-setup --custom-flavor-linters PYTHON_PYLINT,PYTHON_BLACK,PYTHON_FLAKE8,PYTHON_ISORT,PYTHON_BANDIT,PYTHON_MYPY,PYTHON_RUFF,COPYPASTE_JSCPD,JSON_V8R,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_CHECKOV,REPOSITORY_GIT_DIFF,REPOSITORY_GITLEAKS,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

Test Results

    12 files  +    12      12 suites  +12   4h 49m 20s ⏱️ + 4h 49m 20s
 1 139 tests + 1 139   1 139 ✅ + 1 139      0 💤 ±    0  0 ❌ ±0 
13 960 runs  +13 960  10 672 ✅ +10 672  3 288 💤 +3 288  0 ❌ ±0 

Results for commit bf295f6. ± Comparison against base commit d91eaeb.

♻️ This comment has been updated with latest results.

@FlorianPfaff FlorianPfaff merged commit 57e4c67 into main Apr 25, 2026
19 checks passed
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.

1 participant