Skip to content

Split MEAPipeline class out of mea_analysis_routine.py#162

Draft
mandarmp with Copilot wants to merge 2 commits into
mainfrom
copilot/separate-class-and-code
Draft

Split MEAPipeline class out of mea_analysis_routine.py#162
mandarmp with Copilot wants to merge 2 commits into
mainfrom
copilot/separate-class-and-code

Conversation

Copilot AI commented Mar 26, 2026

Copy link
Copy Markdown
Contributor
  • Create _mea_pipeline.py with MEAPipeline class (previous session)
  • Create _mea_runner.py (273 lines) with MEARunOptions, MEARunResult, _normalize_resume_from_stage, _apply_resume_from_stage, run_mea_pipeline — importing from _mea_pipeline.py
  • Slim mea_analysis_routine.py to 311 lines containing only main() (CLI entry point) plus imports from _mea_runner.py and config_loader
  • Remove unreachable _save_checkpoint call after return in _mea_pipeline.py

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI requested a review from mandarmp March 26, 2026 23:03
@mandarmp mandarmp requested a review from adamwea March 26, 2026 23:07
@mandarmp

Copy link
Copy Markdown
Collaborator

Trying to split the mea_analysis_routine into three files @adamwea

@mandarmp mandarmp force-pushed the copilot/separate-class-and-code branch from a2641d0 to be4a359 Compare April 28, 2026 23:10
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