Address review comments

cmake/dependencies.cmake

@@ -75,7 +75,7 @@ if (MUSICA_ENABLE_TUVX AND MUSICA_BUILD_C_CXX_INTERFACE)
   set(TUVX_INSTALL_INCLUDE_DIR ${MUSICA_INSTALL_INCLUDE_DIR} CACHE STRING "" FORCE)
 
   set_git_default(TUVX_GIT_REPOSITORY https://github.com/NCAR/tuv-x.git)
-  set_git_default(TUVX_GIT_TAG f8ae5a2)
+  set_git_default(TUVX_GIT_TAG 0e27463)
 
   FetchContent_Declare(tuvx
     GIT_REPOSITORY ${TUVX_GIT_REPOSITORY}

docker/Dockerfile

@@ -9,7 +9,6 @@ RUN dnf -y update \
         gfortran \
         gdb \
         git \
-        # lapack-devel \
         lcov \
         make \
         netcdf-fortran-devel \

docker/Dockerfile.fortran-gcc

@@ -13,7 +13,6 @@ RUN dnf -y update \
     git \
     hdf5-devel \
     json-devel \
-    # lapack-devel \
     lcov \
     libcurl-devel \
     m4 \

docker/Dockerfile.fortran-gcc.integration

@@ -14,7 +14,6 @@ RUN dnf -y update \
     git \
     hdf5-devel \
     json-devel \
-    # lapack-devel \
     lcov \
     libcurl-devel \
     m4 \

docker/Dockerfile.memcheck

@@ -9,7 +9,6 @@ RUN dnf -y update \
         gfortran \
         gdb \
         git \
-        # lapack-devel \
         lcov \
         make \
         netcdf-fortran-devel \

docker/Dockerfile.mpi

@@ -16,7 +16,6 @@ RUN sudo dnf -y install \
         gcc-c++ \
         gfortran \
         git \
-        # lapack-devel \
         lcov \
         make \
         netcdf-fortran-devel \

docker/Dockerfile.mpi_openmp

@@ -16,7 +16,6 @@ RUN sudo dnf -y install \
         gcc-c++ \
         gfortran \
         git \
-        # lapack-devel \
         lcov \
         make \
         netcdf-fortran-devel \

docker/Dockerfile.openmp

@@ -16,7 +16,6 @@ RUN sudo dnf -y install \
         gcc-c++ \
         gfortran \
         git \
-        # lapack-devel \
         lcov \
         make \
         netcdf-fortran-devel \

docker/Dockerfile.python

@@ -9,7 +9,6 @@ RUN dnf -y update \
         gcc-fortran \
         gdb \
         git \
-        # lapack-devel \
         make \
         netcdf-fortran-devel \
         pip \

include/musica/tuvx/grid.hpp

@@ -14,9 +14,10 @@ namespace musica
   class Profile;
   class Radiator;
 
-  /// @brief A grid struct used to access grid information in tuvx
-  struct Grid
+  /// @brief A grid class used to access grid information in tuvx
+  class Grid
   {
+   public:
     /// @brief Creates a grid instance
     /// @param grid_name The name of the grid
     /// @param units The units of the grid

include/musica/tuvx/grid_map.hpp

@@ -12,9 +12,10 @@
 namespace musica
 {
 
-  /// @brief A grid map struct used to access grid information in tuvx
-  struct GridMap
+  /// @brief A grid map class used to access grid information in tuvx
+  class GridMap
   {
+   public:
     GridMap(void *grid_map)
         : grid_map_(grid_map),
           owns_grid_map_(false)

include/musica/tuvx/profile.hpp

@@ -14,9 +14,10 @@ namespace musica
 {
   class ProfileMap;
 
-  /// @brief A struct used to interact with TUV-x profiles (properties with values on a grid)
-  struct Profile
+  /// @brief A class used to interact with TUV-x profiles (properties with values on a grid)
+  class Profile
   {
+   public:
     /// @brief Creates a profile instance
     /// @param profile_name The name of the profile
     /// @param units The units of the profile

include/musica/tuvx/profile_map.hpp

@@ -13,9 +13,10 @@
 namespace musica
 {
 
-  /// @brief A struct used to store a collection of profiles
-  struct ProfileMap
+  /// @brief A class used to store a collection of profiles
+  class ProfileMap
   {
+   public:
     ProfileMap(void *profile_map)
         : profile_map_(profile_map),
           owns_profile_map_(false)