Skip to content

chore(benchmark): remove longmemeval final answer review file #366

Merged
jinliyl merged 4 commits into
agentscope-ai:mainfrom
jinliyl:dev/golden_result
Jul 16, 2026
Merged

chore(benchmark): remove longmemeval final answer review file #366
jinliyl merged 4 commits into
agentscope-ai:mainfrom
jinliyl:dev/golden_result

Conversation

@jinliyl

@jinliyl jinliyl commented Jul 16, 2026

Copy link
Copy Markdown
Member

No description provided.

jinliyl added 4 commits July 16, 2026 00:44
- Introduce FinalAnswerReviewStep to handle answer validation
- Add final_answer_review.jsonl dataset with 24 evaluation cases
- Include detailed reasoning and golden check results for each case
- Support various question types including temporal reasoning and preferences
- Implement time consistency checks for session references
- Add comprehensive test coverage for different evaluation scenarios
- Removed final_answer_review.jsonl containing 23 evaluation records
- Deleted question_id mappings with detailed reasoning for golden answers
- Removed answer correctness assessments and session time validation checks
- Cleaned up benchmark dataset used for memory evaluation testing
- Eliminated JSONL format evaluation results for temporal reasoning tasks
- Removed references to various session IDs and time-based validations
- Commented out the shell step configuration in default.yaml
- Disabled asynchronous shell command execution capability
- Removed shell step from available backend operations
- Preserved traverse backend configuration unchanged
- Removed test_default_config_registers_shell_job function that was no longer needed
- Kept existing test for frontmatter chunk metadata configuration
- Cleaned up test suite by removing obsolete test case

@ployts ployts left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jinliyl
jinliyl merged commit 2eb0539 into agentscope-ai:main Jul 16, 2026
6 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.

2 participants