Changeset 1509 for XIOS/dev/XIOS_DEV_CMIP6/src/node/domain.cpp

Timestamp:

05/30/18 17:20:39 (6 years ago)

Author:

oabramkina

Message:

Minor bugfix: in certain cases type of XIOS attributes wasn't recognized by the conditional operator ?:.

File:

• XIOS/dev/XIOS_DEV_CMIP6/src/node/domain.cpp (modified) (6 diffs)

XIOS/dev/XIOS_DEV_CMIP6/src/node/domain.cpp

@@ -1801 +1801 @@
              int iend = ibegin + ni -1;
              int jend = jbegin + nj -1;
-             zoom_ibegin = global_zoom_ibegin > ibegin ? global_zoom_ibegin : ibegin;
+             zoom_ibegin = global_zoom_ibegin > ibegin ? global_zoom_ibegin.getValue() : ibegin;
              int zoom_iend  = global_zoom_iend < iend ? zoom_iend : iend ;
              zoom_ni     = zoom_iend-zoom_ibegin+1 ;
 
-             zoom_jbegin = global_zoom_jbegin > jbegin ? global_zoom_jbegin : jbegin ;
+             zoom_jbegin = global_zoom_jbegin > jbegin ? global_zoom_jbegin.getValue() : jbegin ;
              int zoom_jend   = global_zoom_jend < jend ? zoom_jend : jend;
              zoom_nj     = zoom_jend-zoom_jbegin+1;
@@ -2594 +2594 @@
     int zoom_jend_glob = global_zoom_jbegin + global_zoom_nj - 1;
 
-    zoom_ibegin.setValue(global_zoom_ibegin > ibegin ? global_zoom_ibegin : ibegin);
+    zoom_ibegin.setValue(global_zoom_ibegin > ibegin ? global_zoom_ibegin.getValue() : ibegin);
     int zoom_iend = zoom_iend_glob < iend ? zoom_iend_glob : iend ;
     zoom_ni.setValue(zoom_iend-zoom_ibegin+1);
 
-    zoom_jbegin.setValue(global_zoom_jbegin > jbegin ? global_zoom_jbegin : jbegin);
+    zoom_jbegin.setValue(global_zoom_jbegin > jbegin ? global_zoom_jbegin.getValue() : jbegin);
     int zoom_jend = zoom_jend_glob < jend ? zoom_jend_glob : jend ;
     zoom_nj.setValue(zoom_jend-zoom_jbegin+1);
@@ -2657 +2657 @@
   
     if (nbMaskInd != globalLocalIndexMap_.size())
-      info (0) << "If the domain " << this->getDomainOutputName() <<" does not have overlapped region between processes."
-               << "Something must be wrong with mask index "<< std::endl;
+      info (0) << "If domain " << this->getDomainOutputName() <<" does not have overlapped regions between processes "
+               << "something must be wrong with mask index "<< std::endl;
 
     nbMaskInd = globalLocalIndexMap_.size();
@@ -2733 +2733 @@
     
       if (nbLonInd != globalLocalIndexMap_.size())
-        info (0) << "If the domain " << this->getDomainOutputName() <<" does not have overlapped region between processes."
-                 << "Something must be wrong with longitude index "<< std::endl;
+        info (0) << "If domain " << this->getDomainOutputName() <<" does not have overlapped regions between processes "
+                 << "something must be wrong with longitude index "<< std::endl;
 
       nbLonInd = globalLocalIndexMap_.size();
@@ -2817 +2817 @@
     
       if (nbLatInd != globalLocalIndexMap_.size())
-        info (0) << "If the domain " << this->getDomainOutputName() <<" does not have overlapped region between processes."
-                << "Something must be wrong with latitude index "<< std::endl;
+        info (0) << "If domain " << this->getDomainOutputName() <<" does not have overlapped regions between processes "
+                << "something must be wrong with latitude index "<< std::endl;
 
       nbLatInd = globalLocalIndexMap_.size();
@@ -2899 +2899 @@
 
       if (nbAreaInd != globalLocalIndexMap_.size())
-        info (0) << "If the domain " << this->getDomainOutputName() <<" does not have overlapped region between processes."
-                 << "Something must be wrong with area index "<< std::endl;
+        info (0) << "If domain " << this->getDomainOutputName() <<" does not have overlapped regions between processes "
+                 << "something must be wrong with area index "<< std::endl;
 
       nbAreaInd = globalLocalIndexMap_.size();