Seems to work

openPMD · Jul 10, 2024 · 88c991a · 88c991a
1 parent 1713314
commit 88c991a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/src/IO/IOTask.cpp b/src/IO/IOTask.cpp
@@ -87,8 +87,8 @@ json::ParsedConfig Parameter<Operation::CREATE_DATASET>::compileJSONConfig(
     return json::ParsedConfig{
         std::move(base_config.config),
         (options.empty() || options == "{}")
-            ? manual_config.originallySpecifiedAs
-            : base_config.originallySpecifiedAs};
+            ? base_config.originallySpecifiedAs
+            : manual_config.originallySpecifiedAs};
 }
 
 namespace internal

diff --git a/src/Series.cpp b/src/Series.cpp
@@ -1085,7 +1085,7 @@ void Series::initSeries(
     }
 
     series.iterations.linkHierarchy(writable);
-    series.iterations.writable().ownKeyWithinParent = "iterations";
+    series.iterations.writable().ownKeyWithinParent = "data";
     series.m_rankTable.m_attributable.linkHierarchy(writable);
 
     series.m_name = input->name;

diff --git a/src/backend/Attributable.cpp b/src/backend/Attributable.cpp
@@ -195,17 +195,14 @@ std::string Attributable::MyPath::openPMDPath() const
 {
     if (group.empty())
     {
-        return std::string();
+        return std::string("/");
     }
     else
     {
         std::stringstream res;
-        auto it = group.begin();
-        auto end = group.end();
-        res << *it++;
-        for (; it != end; ++it)
+        for (auto const &element : group)
         {
-            res << '/' << *it;
+            res << '/' << element;
         }
         return res.str();
     }