From 744039eb2f77eda5e2a025624dde0ab576113589 Mon Sep 17 00:00:00 2001
From: Joakim Hove <joakim.hove@gmail.com>
Date: Sun, 9 May 2021 11:54:45 +0200
Subject: [PATCH] Add documentation to WellContainer<T> class

---
 opm/simulators/wells/WellContainer.hpp | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/opm/simulators/wells/WellContainer.hpp b/opm/simulators/wells/WellContainer.hpp
index 259daa1e7..c230e9818 100644
--- a/opm/simulators/wells/WellContainer.hpp
+++ b/opm/simulators/wells/WellContainer.hpp
@@ -27,6 +27,19 @@
 
 namespace Opm {
 
+
+/*
+  The WellContainer<T> class is a small utility class designed to manage the
+  dynamic state of per well quantities, like active control and phase rates. The
+  values are stored continously in a vector, but they are added with a name, and
+  can also be accessed and updated with the name.
+
+  The class is created to facilitate safe and piecewise refactoring of the
+  WellStateFullyImplicitBlackOil class, and might have a short life in the
+  development timeline.
+*/
+
+
 template <class T>
 class WellContainer {
 public:
@@ -57,6 +70,10 @@ public:
         this->data.at(index) = std::forward<T>(value);
     }
 
+    /*
+      Will copy the value from other to this - for all wells which are present
+      in both containers.
+    */
     void copy_welldata(const WellContainer<T>& other) {
         if (this->index_map == other.index_map)
             this->data = other.data;
@@ -66,6 +83,10 @@ public:
         }
     }
 
+    /*
+      Will copy the value for well @name from other to this. The well @name must
+      exist in both containers, otherwise an exception is thrown.
+    */
     void copy_welldata(const WellContainer<T>& other, const std::string& name) {
         auto this_index = this->index_map.at(name);
         auto other_index = other.index_map.at(name);