The other changes in this file use "if target_location_namespace", and here it's "if target_location_namespace is not None".  Is the difference important?  If not, pick the shorter.   Also, not a great line-break...