I see that this is correct, but the semantics of increment_subtask_status was confusing at first.  (Mainly from seeing a hardcoded 1)