Skip to content

Refactor evaluation dataset preparation helper#180

Closed
benjibc wants to merge 4 commits intomainfrom
codex/refactor-evaluation_test-decorator-function
Closed

Refactor evaluation dataset preparation helper#180
benjibc wants to merge 4 commits intomainfrom
codex/refactor-evaluation_test-decorator-function

Conversation

@benjibc
Copy link
Copy Markdown
Contributor

@benjibc benjibc commented Sep 16, 2025

Summary

  • extract dataset loading and row-id preparation from evaluation_test into a dedicated load_and_prepare_rows helper
  • update evaluation_test to use the new helper for clarity
  • add targeted unit tests for the helper that cover dataset paths, raw messages, and pre-built rows while stubbing optional deps

Testing

  • pytest tests/pytest/test_dataset_preparation.py

https://chatgpt.com/codex/tasks/task_e_68c8edc63cc48333bc4942d89d0e2035

@benjibc benjibc force-pushed the codex/refactor-evaluation_test-decorator-function branch from 30ed9b0 to 44ec364 Compare September 16, 2025 21:32
@benjibc benjibc closed this Sep 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant