A couple comments around/within this block would be nice, explaining the conditions it should happen under. It took me a couple reads to understand why the final statement wasn't dedented a level (seems like you *shouldn't* be able to exit this method without setting ``self._models_loaded = True``, until understanding that ``populate_models`` may have been called one or more times previously in the stack).