From e952090de92bef5cdb7c14b0aa2119efeaf99f6a Mon Sep 17 00:00:00 2001
From: Ben Dudson <dudson2@llnl.gov>
Date: Wed, 11 Mar 2026 11:06:15 -0700
Subject: [PATCH 1/4] Options: Convert Tensor<int> to Tensor<BoutReal> and
 similar

Read Tensor<int> from NetCDF files, and allow conversion to
Tensor<BoutReal> and then Field3D.
---
 include/bout/array.hxx             |  8 ++++++++
 include/bout/utils.hxx             | 18 +++++++++++++++++-
 src/sys/options.cxx                | 29 ++++++++++++++++++++++++-----
 src/sys/options/options_netcdf.cxx |  6 ++++++
 4 files changed, 55 insertions(+), 6 deletions(-)

diff --git a/include/bout/array.hxx b/include/bout/array.hxx
index 2305fe3b22..000ded9616 100644
--- a/include/bout/array.hxx
+++ b/include/bout/array.hxx
@@ -227,6 +227,14 @@ public:
     ptr = get(new_size);
   }
 
+  /*!
+   * Change shape of the container.
+   * Invalidates contents.
+   */
+  void reshape(std::tuple<size_type> new_shape) {
+    reallocate(std::get<0>(new_shape));
+  }
+
   /*!
    * Holds a static variable which controls whether
    * memory blocks (dataBlock) are put into a store
diff --git a/include/bout/utils.hxx b/include/bout/utils.hxx
index 560f97f954..5088a025c1 100644
--- a/include/bout/utils.hxx
+++ b/include/bout/utils.hxx
@@ -5,7 +5,7 @@
  * simple but common calculations
  *
  **************************************************************************
- * Copyright 2010 - 2025 BOUT++ contributors
+ * Copyright 2010 - 2026 BOUT++ contributors
  *
  * Contact: Ben Dudson, dudson2@llnl.gov
  * 
@@ -231,6 +231,14 @@ public:
     data.reallocate(new_size_1 * new_size_2);
   }
 
+  /*!
+   * Change shape of the container.
+   * Invalidates contents.
+   */
+  void reshape(std::tuple<size_type, size_type> new_shape) {
+    reallocate(std::get<0>(new_shape), std::get<1>(new_shape));
+  }
+
   Matrix& operator=(const Matrix& other) {
     n1 = other.n1;
     n2 = other.n2;
@@ -331,6 +339,14 @@ public:
     data.reallocate(new_size_1 * new_size_2 * new_size_3);
   }
 
+  /*!
+   * Change shape of the container.
+   * Invalidates contents.
+   */
+  void reshape(std::tuple<size_type, size_type, size_type> new_shape) {
+    reallocate(std::get<0>(new_shape), std::get<1>(new_shape), std::get<2>(new_shape));
+  }
+
   Tensor& operator=(const Tensor& other) {
     n1 = other.n1;
     n2 = other.n2;
diff --git a/src/sys/options.cxx b/src/sys/options.cxx
index d60e91079d..8bd83c0bfb 100644
--- a/src/sys/options.cxx
+++ b/src/sys/options.cxx
@@ -744,27 +744,46 @@ FieldPerp Options::as<FieldPerp>(const FieldPerp& similar_to) const {
 }
 
 namespace {
-/// Visitor to convert an int, BoutReal or Array/Matrix/Tensor to the
-/// appropriate container
+/// Primary declaration of ConvertContainer, for specialization below.
+/// No definition needed unless it is used.
 template <class Container>
-struct ConvertContainer {
+struct ConvertContainer;
+
+/// Visitor to convert an int, BoutReal or Array/Matrix/Tensor to the
+/// appropriate container. Templated on both the container class C
+/// and scalar type Scalar.
+template <template <class> class C, class Scalar>
+struct ConvertContainer<C<Scalar>> {
+  using Container = C<Scalar>;
   ConvertContainer(std::string error, Container similar_to_)
       : error_message(std::move(error)), similar_to(std::move(similar_to_)) {}
 
   Container operator()(int value) {
     Container result(similar_to);
-    std::fill(std::begin(result), std::end(result), value);
+    std::fill(std::begin(result), std::end(result), static_cast<Scalar>(value));
     return result;
   }
 
   Container operator()(BoutReal value) {
     Container result(similar_to);
-    std::fill(std::begin(result), std::end(result), value);
+    std::fill(std::begin(result), std::end(result), static_cast<Scalar>(value));
     return result;
   }
 
   Container operator()(const Container& value) { return value; }
 
+  // Convert between scalar types: C<OtherScalar> -> C<Scalar>
+  // The size of the returned result will be the same as the input value
+  template <class OtherScalar>
+  Container operator()(const C<OtherScalar>& value) {
+    Container result(similar_to);
+    result.reshape(value.shape()); // Resize to shape of input
+
+    std::transform(std::begin(value), std::end(value), std::begin(result),
+                   [](const OtherScalar& x) { return static_cast<Scalar>(x); });
+    return result;
+  }
+
   template <class Other>
   Container operator()([[maybe_unused]] const Other& value) {
     throw BoutException(error_message);
diff --git a/src/sys/options/options_netcdf.cxx b/src/sys/options/options_netcdf.cxx
index d3e5a993bd..f21f461c09 100644
--- a/src/sys/options/options_netcdf.cxx
+++ b/src/sys/options/options_netcdf.cxx
@@ -158,6 +158,12 @@ void readGroup(const std::string& filename, const NcGroup& group, Options& resul
           var.getVar(value.begin());
           result[var_name] = value;
         }
+      } else if (var_type == ncInt) {
+        Tensor<int> value(static_cast<int>(dims[0].getSize()),
+                          static_cast<int>(dims[1].getSize()),
+                          static_cast<int>(dims[2].getSize()));
+        var.getVar(value.begin());
+        result[var_name] = value;
       }
     }
     }

From b2e2f3575e68f771be2a4341af5fd14e2870469e Mon Sep 17 00:00:00 2001
From: bendudson <219233+bendudson@users.noreply.github.com>
Date: Wed, 11 Mar 2026 18:11:01 +0000
Subject: [PATCH 2/4] [bot] Apply format changes

---
 include/bout/array.hxx | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/include/bout/array.hxx b/include/bout/array.hxx
index 000ded9616..82677fd38a 100644
--- a/include/bout/array.hxx
+++ b/include/bout/array.hxx
@@ -231,9 +231,7 @@ public:
    * Change shape of the container.
    * Invalidates contents.
    */
-  void reshape(std::tuple<size_type> new_shape) {
-    reallocate(std::get<0>(new_shape));
-  }
+  void reshape(std::tuple<size_type> new_shape) { reallocate(std::get<0>(new_shape)); }
 
   /*!
    * Holds a static variable which controls whether

From 05e5f138691bcaff6c890abfc5e2fdc0a7f8dfb9 Mon Sep 17 00:00:00 2001
From: bendudson <219233+bendudson@users.noreply.github.com>
Date: Wed, 11 Mar 2026 18:11:02 +0000
Subject: [PATCH 3/4] [bot] Add last format changes commit to ignore file

---
 .git-blame-ignore-revs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.git-blame-ignore-revs b/.git-blame-ignore-revs
index cc795d6d86..33bf3dd3fb 100644
--- a/.git-blame-ignore-revs
+++ b/.git-blame-ignore-revs
@@ -17,3 +17,4 @@ d1cfb8abd6aa5c76e6c1a4d7ab20929c65f8afc2
 ec69e8838be2dde140a915e50506f8e1ce3cb534
 f2bc0488a298f136294c523bc5ab4086d090059b
 1b4707187a3a85126338303dc766280b8fb2dc56
+b2e2f3575e68f771be2a4341af5fd14e2870469e

From 7c5d9d8a36485d0ce79fa03c255bb57fb2acb559 Mon Sep 17 00:00:00 2001
From: Ben Dudson <dudson2@llnl.gov>
Date: Wed, 11 Mar 2026 13:50:49 -0700
Subject: [PATCH 4/4] Options: Unit test of Tensor<int> to Tensor<BoutReal>

---
 tests/unit/sys/test_options.cxx | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/tests/unit/sys/test_options.cxx b/tests/unit/sys/test_options.cxx
index fb3e6a62c5..d4d803d685 100644
--- a/tests/unit/sys/test_options.cxx
+++ b/tests/unit/sys/test_options.cxx
@@ -1470,3 +1470,36 @@ TEST_F(OptionsTest, MatrixInt) {
   const auto matrix_out = options["int_matrix"].as<Matrix<int>>();
   ASSERT_EQ(matrix_out(nx - 1, ny - 1), matrix_in(nx - 1, ny - 1));
 }
+
+TEST_F(OptionsTest, TensorIntToTensorBoutReal) {
+  constexpr int nx = 2;
+  constexpr int ny = 3;
+  constexpr int nz = 4;
+
+  Tensor<int> t_int(nx, ny, nz);
+  int count = 0;
+  for (int i = 0; i < nx; ++i) {
+    for (int j = 0; j < ny; ++j) {
+      for (int k = 0; k < nz; ++k) {
+        t_int(i, j, k) = ++count;
+      }
+    }
+  }
+
+  Options option = t_int;
+
+  // Convert to Tensor<BoutReal>
+  Tensor<BoutReal> t_boutreal = option.as<Tensor<BoutReal>>();
+
+  std::tuple<int, int, int> expected_shape{nx, ny, nz};
+  ASSERT_EQ(expected_shape, t_boutreal.shape());
+
+  count = 0;
+  for (int i = 0; i < nx; ++i) {
+    for (int j = 0; j < ny; ++j) {
+      for (int k = 0; k < nz; ++k) {
+        ASSERT_FLOAT_EQ(t_boutreal(i, j, k), static_cast<BoutReal>(++count));
+      }
+    }
+  }
+}