Changeset 1315 for XIOS/trunk

Timestamp:

10/25/17 16:36:53 (7 years ago)

Author:

rlacroix

Message:

Fix the default "freq_offset" for fields read from files.

Also throw an error when trying to use an invalid operation for such fields (currently only "instant" are supported).

File:

• XIOS/trunk/src/node/field.cpp (modified) (8 diffs)

XIOS/trunk/src/node/field.cpp

@@ -788 +788 @@
      if (!areAllReferenceSolved) solveAllReferenceEnabledField(false);
 
+     checkAttributes();
+
      const bool detectMissingValues = (!detect_missing_value.isEmpty() && !default_value.isEmpty() && detect_missing_value == true);
      const double defaultValue  = detectMissingValues ? default_value : (!default_value.isEmpty() ? default_value : 0.0);
@@ -821 +823 @@
        // Check if the data is to be read from a file
        else if (file && !file->mode.isEmpty() && file->mode == CFile::mode_attr::read)
-         instantDataFilter = serverSourceFilter = boost::shared_ptr<CSourceFilter>(new CSourceFilter(gc, grid,
-                                                                                                     freq_offset.isEmpty() ? NoneDu : freq_offset,
-                                                                                                     true,
+         instantDataFilter = serverSourceFilter = boost::shared_ptr<CSourceFilter>(new CSourceFilter(gc, grid, freq_offset, true,
                                                                                                      detectMissingValues, defaultValue));
        else // The data might be passed from the model
@@ -900 +900 @@
      if (!selfReferenceFilter)
      {
+       checkAttributes();
+
        const bool detectMissingValues = (!detect_missing_value.isEmpty() && !default_value.isEmpty() && detect_missing_value == true);
        const double defaultValue  = detectMissingValues ? default_value : (!default_value.isEmpty() ? default_value : 0.0);
@@ -906 +908 @@
        {
          if (!serverSourceFilter)
-           serverSourceFilter = boost::shared_ptr<CSourceFilter>(new CSourceFilter(gc, grid,
-                                                                                   freq_offset.isEmpty() ? NoneDu : freq_offset,
-                                                                                   true,
+           serverSourceFilter = boost::shared_ptr<CSourceFilter>(new CSourceFilter(gc, grid, freq_offset, true,
                                                                                    detectMissingValues, defaultValue));
 
@@ -954 +954 @@
                << "An operation must be defined for field \"" << getId() << "\".");
 
+       checkAttributes();
+
        const bool detectMissingValues = (!detect_missing_value.isEmpty() && !default_value.isEmpty() && detect_missing_value == true);
-       checkAttributes() ;
        boost::shared_ptr<CTemporalFilter> temporalFilter(new CTemporalFilter(gc, operation,
                                                                              CContext::getCurrent()->getCalendar()->getInitDate(),
@@ -991 +992 @@
                << "An operation must be defined for field \"" << getId() << "\".");
 
+       checkAttributes();
+
        const bool detectMissingValues = (!detect_missing_value.isEmpty() && !default_value.isEmpty() && detect_missing_value == true);
-       checkAttributes();
        boost::shared_ptr<CTemporalFilter> temporalFilter(new CTemporalFilter(gc, operation,
                                                                              CContext::getCurrent()->getCalendar()->getInitDate(),
@@ -1442 +1444 @@
    void CField::checkAttributes(void)
    {
+     bool isFieldRead = file && !file->mode.isEmpty() && file->mode == CFile::mode_attr::read;
+     if (isFieldRead && operation.getValue() != "instant")
+       ERROR("void CField::checkAttributes(void)",
+             << "Unsupported operation for field '" << getFieldOutputName() << "'." << std::endl
+             << "Currently only \"instant\" is supported for fields read from file.")
+
      if (freq_op.isEmpty())
      {
-       if (operation.getValue()=="instant")
+       if (operation.getValue() == "instant")
          freq_op.setValue(file->output_freq.getValue());
        else
@@ -1450 +1458 @@
      }
      if (freq_offset.isEmpty())
-       freq_offset.setValue(freq_op.getValue()-TimeStep);
+       freq_offset.setValue(isFieldRead ? NoneDu : (freq_op.getValue() - TimeStep));
    }