Work Done
Reviewed only the staged diff on feat/isup-inference. Checked repository-wide staged imports, documentation references, package/module naming, restored behavior against origin/unify-vits and origin/neerav/isup-two-stage, and simulated merges in a temporary clone. Found that the restored EMA path omits output processors present on the two-stage branch and that the README config tree omits the restored inference config. Confirmed both comparison branches merge without textual conflicts and found no stale staged imports or Python package collision.
Lessons Learned: Pitfalls
The infrastructure workstream material was not relevant to this narrow code review. The local Pixi development environment supports Linux only, so lint and import checks could not run on the macOS host.
Lessons Learned: Improvements
A short ve2e development SOP that states which branches contain required inference behavior fixes and describes a host-independent staged-tree validation command would speed future cross-branch reviews.