Skip to content

Fix option matching without cushion - #2576

Open
Danielxu0208 wants to merge 1 commit into
open-compass:mainfrom
Danielxu0208:tech/fix-option-postprocessor
Open

Fix option matching without cushion#2576
Danielxu0208 wants to merge 1 commit into
open-compass:mainfrom
Danielxu0208:tech/fix-option-postprocessor

Conversation

@Danielxu0208

Copy link
Copy Markdown

Summary

  • restore the missing separator between two Chinese answer patterns
  • capture the option rather than its leading whitespace in the generic boundary pattern
  • add regression coverage with cushion=False

Why

Two adjacent f-strings were implicitly concatenated, making the intended 只有选项 ... 是对 pattern unreachable. A later fallback pattern also captured the boundary as group 1, so matches such as reason A because were discarded when cushion matching was disabled.

Validation

  • targeted checks for 只有选项 B 是对 and reason A because
  • existing 答案是 B behavior retained
  • git diff --check

@Danielxu0208
Danielxu0208 force-pushed the tech/fix-option-postprocessor branch from 24b2302 to 4fd8f87 Compare August 3, 2026 08:18
@Danielxu0208
Danielxu0208 marked this pull request as ready for review August 6, 2026 16:11
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.

2 participants