Skip to content

SEACAS: cgns - fix zgc uniquification

James Willenbring requested to merge gsjaardema:seacas-cgns-zgc-fix into develop

Created by: gsjaardema

@trilinos/seacas

Description

Fix for SPARC

  • The ZoneGridConnectivity instances need to be unique for a zone. In the file-per-processor case, there may be multiple ZGC with the same name that arise due to processor decomposition. The previous code assumed a maximum of 26 instances and made them unique by appending a 'A' ... 'Z' to the name until found a unique name. Failure was not detected.

New code tries 'A' .. 'Z' and if not unique, then tries 'AA' 'AB',...'ZZ' and if still not unique it will stop and output a meaningful error message.

Also added some debugging output for the non-file-per-processor output to make sure that zgc consolidation is working correctly

Motivation and Context

How Has This Been Tested?

Tested on actual mesh that was causing issues with original code. A representation of this mesh is now in test suite (which will be in later commit)

Checklist

  • My commit messages mention the appropriate GitHub issue numbers.
  • My code follows the code style of the affected package(s).
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the code contribution guidelines for this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • No new compiler warnings were introduced.
  • These changes break backwards compatibility.

Merge request reports