While exploring #1646, we realized that there are various internal calls to to_unstructured that only care about the topology output. Since to_unstructured always creates a coordset, even if it will just be thrown away afterwards, we think it would be better to explore overloading the API to have a version of to_unstructured that skips coordset creation entirely by taking the coordset name as a string. 1) This is more efficient and 2) it alleviates the problem of needing a strange pattern for calling to_unstructured to ensure that a coordset name is always set.
While exploring #1646, we realized that there are various internal calls to
to_unstructuredthat only care about the topology output. Sinceto_unstructuredalways creates a coordset, even if it will just be thrown away afterwards, we think it would be better to explore overloading the API to have a version ofto_unstructuredthat skips coordset creation entirely by taking the coordset name as a string. 1) This is more efficient and 2) it alleviates the problem of needing a strange pattern for callingto_unstructuredto ensure that a coordset name is always set.