Skip to content

Cleanup

Stephan Rayner requested to merge cleanup into main

What does this merge request do and why?

Our code base has gotten a little messy as we have built it. I have taken it upon myself to clean it up.

  • Removing defaults that cause problems (like me loosing a week of my life)
  • Making CLI flags explicit (related to above point)
  • Consolidating the batch completion within the beam portion of the pipeline

The list will go on.

Merge request checklist

  • I've ran the eval_codebase.py pipeline to validate that nothing is broken.
  • Tests added for new functionality. If not, please raise an issue to follow up.
    • Functionality remains unchanged but I will see I actually need more tests.
  • Documentation added/updated, if needed
    • Nothing from how you run this should change and if your script breaks now it will tell you
Edited by Stephan Rayner

Merge request reports