fix(scicode): read assistant text blocks#1381
Conversation
There was a problem hiding this comment.
Code Review
This pull request simplifies the code extraction logic in evalscope/benchmarks/scicode/util.py by accessing message.text directly instead of casting message.content to a string. It also adds a unit test in tests/benchmark/test_scicode_util.py to verify that get_generated_code correctly handles assistant messages with text content blocks. There are no review comments to address, and the changes appear correct and complete.
95ff96d to
dad292d
Compare
Yunnglin
left a comment
There was a problem hiding this comment.
LGTM. Fixed a minor yapf formatting issue and pushed the fix.
|
I pushed a small formatting fix (yapf) to your branch — the list comprehension in |
Summary
Fixes #1380.
To verify