IOSS: Default to not munging block/set names on database

Original behavior was to convert all database block/set names to
lowercase and convert spaces to '_'.  The new default is proposed to
be leaving the database names as they are read from the database and
adding an alias of the name lowercased.

Added property `LOWER_CASE_DATABASE_NAMES` which is false by default.
If true, then previous behavior is returned.

io_shell has --lower_case_database_names option to also select that
behavior for testing.

Author: gsjaardema

packages/seacas/libraries/ioss/src/Ioss_DatabaseIO.C

@@ -347,6 +347,7 @@ namespace Ioss {
     }
 
     Utils::check_set_bool_property(properties, "LOWER_CASE_VARIABLE_NAMES", lowerCaseVariableNames);
+    Utils::check_set_bool_property(properties, "LOWER_CASE_DATABASE_NAMES", lowerCaseDatabaseNames);
     Utils::check_set_bool_property(properties, "USE_GENERIC_CANONICAL_NAMES",
                                    useGenericCanonicalName);
     Utils::check_set_bool_property(properties, "IGNORE_DATABASE_NAMES", ignoreDatabaseNames);

packages/seacas/libraries/ioss/src/Ioss_DatabaseIO.h

@@ -1,4 +1,4 @@
-// Copyright(C) 1999-2024 National Technology & Engineering Solutions
+// Copyright(C) 1999-2025 National Technology & Engineering Solutions
 // of Sandia, LLC (NTESS).  Under the terms of Contract DE-NA0003525 with
 // NTESS, the U.S. Government retains certain rights in this software.
 //
@@ -531,6 +531,11 @@ namespace Ioss {
       lowerCaseVariableNames = true_false;
     }
 
+    void set_lower_case_database_names(bool true_false) const
+    {
+      lowerCaseDatabaseNames = true_false;
+    }
+
     /* \brief Set the method used to split sidesets into homogeneous blocks.
      *
      *  \param[in] split_type The desired method.
@@ -767,6 +772,7 @@ namespace Ioss {
     bool         isParallel{false}; //!< true if running in parallel
 
     mutable bool lowerCaseVariableNames{true};
+    mutable bool lowerCaseDatabaseNames{false};
     bool         usingParallelIO{false};
 
     // List of element blocks that should be omitted or included from

packages/seacas/libraries/ioss/src/Ioss_Doxygen.h

@@ -47,6 +47,7 @@ par_cgns         | Input/Output  | alias for parallel CGNS
  ----------|:--------:|------------
  LOGGING   | on/[off] | enable/disable logging of field input/output
  LOWER_CASE_VARIABLE_NAMES | [on]/off | Convert all variable names read from input database to lowercase; replace ' ' with '_'
+ LOWER_CASE_DATABASE_NAMES | on/[off] | Convert all block/set names read from input database to lowercase; replace ' ' with '_'
  USE_GENERIC_CANONICAL_NAMES | on/[off]  | use `block_{id}` as canonical name of an element block instead of the name (if any) stored on the database. The database name will be an alias.
  IGNORE_DATABASE_NAMES | on/[off] | Do not read any element block, nodeset, ... names if they exist on the database.  Use only the canonical generated names (entitytype + _ + id)
  IGNORE_ATTRIBUTE_NAMES   | on/[off] | Do not read the attribute names that may exist on an input database. Instead for an element block with N attributes, the fields will be named `attribute_1` ... `attribute_N`

packages/seacas/libraries/ioss/src/Ioss_EntityBlock.C

@@ -1,4 +1,4 @@
-// Copyright(C) 1999-2023 National Technology & Engineering Solutions
+// Copyright(C) 1999-2023, 2025 National Technology & Engineering Solutions
 // of Sandia, LLC (NTESS).  Under the terms of Contract DE-NA0003525 with
 // NTESS, the U.S. Government retains certain rights in this software.
 //

packages/seacas/libraries/ioss/src/Ioss_Region.C

@@ -1744,6 +1744,11 @@ namespace Ioss {
         aliases_[type].insert(std::make_pair(uname, canon));
       }
 
+      std::string lname = Ioss::Utils::lowercase(alias);
+      if (lname != alias && lname != canon) {
+        aliases_[type].insert(std::make_pair(lname, canon));
+      }
+
       bool result;
       std::tie(std::ignore, result) = aliases_[type].insert(std::make_pair(alias, canon));
       return result;

packages/seacas/libraries/ioss/src/Ioss_Version.h

@@ -6,5 +6,5 @@
 
 #pragma once
 namespace Ioss {
-  inline const char *Version() { return "2025-03-13"; }
+  inline const char *Version() { return "2025-04-14"; }
 } // namespace Ioss

packages/seacas/libraries/ioss/src/exodus/Ioex_DatabaseIO.C

@@ -1268,7 +1268,7 @@ namespace Ioex {
       else {
         Ioss::SerializeIO serializeio_(this);
         block_name = Ioex::get_entity_name(get_file_pointer(), entity_type, id, basename,
-                                           maximumNameLength, db_has_name);
+                                           maximumNameLength, lowerCaseDatabaseNames, db_has_name);
       }
       if (get_use_generic_canonical_name()) {
         std::swap(block_name, alias);
@@ -1597,7 +1597,7 @@ namespace Ioex {
           }
           else {
             side_set_name = Ioex::get_entity_name(get_file_pointer(), EX_SIDE_SET, id, "surface",
-                                                  maximumNameLength, db_has_name);
+                                                  maximumNameLength, lowerCaseDatabaseNames, db_has_name);
           }
 
           if (side_set_name == "universal_sideset") {
@@ -2002,7 +2002,7 @@ namespace Ioex {
           }
           else {
             Xset_name = Ioex::get_entity_name(get_file_pointer(), type, id, base + "list",
-                                              maximumNameLength, db_has_name);
+                                              maximumNameLength, lowerCaseDatabaseNames, db_has_name);
           }
 
           if (get_use_generic_canonical_name()) {

packages/seacas/libraries/ioss/src/exodus/Ioex_Internals.C

@@ -1,4 +1,4 @@
-// Copyright(C) 1999-2024 National Technology & Engineering Solutions
+// Copyright(C) 1999-2025 National Technology & Engineering Solutions
 // of Sandia, LLC (NTESS).  Under the terms of Contract DE-NA0003525 with
 // NTESS, the U.S. Government retains certain rights in this software.
 //

packages/seacas/libraries/ioss/src/exodus/Ioex_Utils.C

@@ -658,7 +658,7 @@ namespace Ioex {
   }
 
   std::string get_entity_name(int exoid, ex_entity_type type, int64_t id,
-                              const std::string &basename, int length, bool &db_has_name)
+                              const std::string &basename, int length, bool lower_case_names, bool &db_has_name)
   {
     std::vector<char> buffer(length + 1);
     buffer[0] = '\0';
@@ -667,16 +667,18 @@ namespace Ioex {
       exodus_error(exoid, __LINE__, __func__, __FILE__);
     }
     if (buffer[0] != '\0') {
-      Ioss::Utils::fixup_name(Data(buffer));
+      std::string name{Data(buffer)};
+      if (lower_case_names) {
+	Ioss::Utils::fixup_name(name);
+      }
       // Filter out names of the form "basename_id" if the name
       // id doesn't match the id in the name...
-      size_t base_size = basename.size();
-      if (std::strncmp(basename.c_str(), Data(buffer), base_size) == 0) {
-        int64_t name_id = extract_id(Data(buffer));
+      if (Ioss::Utils::substr_equal(basename, name)) {
+	int64_t name_id = extract_id(name);
 
         // See if name is truly of form "basename_name_id" (e.g. "surface_{id}")
         std::string tmp_name = Ioss::Utils::encode_entity_name(basename, name_id);
-        if (tmp_name == Data(buffer)) {
+        if (tmp_name == name) {
           if (name_id > 0) {
             db_has_name = false;
             if (name_id != id) {
@@ -686,15 +688,15 @@ namespace Ioex {
                          "embedded id {}.\n"
                          "         This can cause issues later; the entity will be renamed to '{}' "
                          "(IOSS)\n\n",
-                         Data(buffer), id, name_id, new_name);
+                         name, id, name_id, new_name);
               return new_name;
             }
             return tmp_name;
           }
         }
       }
       db_has_name = true;
-      return {Data(buffer)};
+      return name;
     }
     db_has_name = false;
     return Ioss::Utils::encode_entity_name(basename, id);

packages/seacas/libraries/ioss/src/exodus/Ioex_Utils.h

@@ -126,7 +126,7 @@ namespace Ioex {
 
   IOSS_NODISCARD IOEX_EXPORT std::string get_entity_name(int exoid, ex_entity_type type, int64_t id,
                                                          const std::string &basename, int length,
-                                                         bool &db_has_name);
+                                                         bool lower_case_names, bool &db_has_name);
 
   IOEX_EXPORT bool filter_node_list(Ioss::Int64Vector                &nodes,
                                     const std::vector<unsigned char> &node_connectivity_status);