You don't need to change this to merge this PR, but in the future try to use messages like:

raise NotImplementedError(str(type(self))+" does not implement get_output_space.")

The python interpreter doesn't tell you what derived class failed to implement the method, so this can be a pain to debug if the error message itself doesn't tell you.