Sam, why is self redundant here but not in set_module_progress?