Merge pull request #1152 from arcaneframework/dev/gg-add-hash-only-in-basic-writer

Add support for saving only comparison hash in 'BasicWriter'

Author: grospelliergilles

arcane/src/arcane/core/IVerifierService.h

@@ -1,21 +1,20 @@
 // -*- tab-width: 2; indent-tabs-mode: nil; coding: utf-8-with-signature -*-
 //-----------------------------------------------------------------------------
-// Copyright 2000-2022 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
+// Copyright 2000-2024 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
 // See the top-level COPYRIGHT file for details.
 // SPDX-License-Identifier: Apache-2.0
 //-----------------------------------------------------------------------------
 /*---------------------------------------------------------------------------*/
-/* IVerifierService.h                                          (C) 2000-2022 */
+/* IVerifierService.h                                          (C) 2000-2024 */
 /*                                                                           */
 /* Interface du service de vérification des données entre deux exécutions.   */
 /*---------------------------------------------------------------------------*/
-#ifndef ARCANE_IVERIFIERSERVICE_H
-#define ARCANE_IVERIFIERSERVICE_H
+#ifndef ARCANE_CORE_IVERIFIERSERVICE_H
+#define ARCANE_CORE_IVERIFIERSERVICE_H
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
-#include "arcane/ArcaneTypes.h"
-#include "arcane/IService.h"
+#include "arcane/core/ArcaneTypes.h"
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
@@ -32,17 +31,38 @@ namespace Arcane
  */
 class IVerifierService
 {
+ public:
+
+  /*!
+   * \brief Mode de comparaison
+   */
+  enum class eCompareMode
+  {
+    //! Compare toutes les valeurs
+    Values,
+    /*!
+     * \brief Compare uniquement les hash des valeurs.
+     *
+     * Ce mode permet uniquement de détecter si deux valeurs sont différentes
+     * sans pouvoir connaitre cette différence. Il est cependant plus rapide
+     * que le mode \a Values et permet de limiter la taille des fichiers
+     * de comparaison.
+     */
+    HashOnly
+  };
+
  public:
 
   //! Libère les ressources
-  virtual ~IVerifierService() {}
+  virtual ~IVerifierService() = default;
 
  public:
 
   //! Ecrit le fichier référence
-  virtual void writeReferenceFile() =0;
+  virtual void writeReferenceFile() = 0;
 
-  /*! \brief Effectue la vérification à partir du fichier référence.
+  /*!
+   * \brief Effectue la vérification à partir du fichier référence.
    *
    * \param parallel_sequential si vrai, indique qu'on compare le résultat
    * d'une exécution parallèle avec celui d'une exécution séquentielle. Cette
@@ -55,26 +75,31 @@ class IVerifierService
    * en général ne pas faire de vérification sur les entités fantômes. Cette
    * option est inactive si l'exécution est séquentielle.
    */
-  virtual void doVerifFromReferenceFile(bool parallel_sequential,bool compare_ghost) =0;
+  virtual void doVerifFromReferenceFile(bool parallel_sequential, bool compare_ghost) = 0;
 
  public:
-  
+
   //! Positionne le nom du fichier contenant les valeurs de référence
-  virtual void setFileName(const String& file_name) =0;
+  virtual void setFileName(const String& file_name) = 0;
   //! Nom du fichier contenant les valeurs de référence
-  virtual String fileName() const =0;
+  virtual String fileName() const = 0;
 
  public:
 
-  virtual void setResultFileName(const String& file_name) =0;
-  virtual String resultfileName() const =0;
+  //! Nom du fichier contenant les résultats
+  virtual void setResultFileName(const String& file_name) = 0;
+  virtual String resultfileName() const = 0;
+
+  //! Type de comparaison souhaité
+  virtual void setCompareMode(eCompareMode v) = 0;
+  virtual eCompareMode compareMode() const = 0;
 
  public:
-  
+
   //! Positionne le nom du sous répertoire contenant les valeurs de référence
-  virtual void setSubDir(const String& sub_dir) =0;
+  virtual void setSubDir(const String& sub_dir) = 0;
   //! Nom du fichier contenant les valeurs de référence
-  virtual String subDir() const =0;
+  virtual String subDir() const = 0;
 };
 
 /*---------------------------------------------------------------------------*/

arcane/src/arcane/core/VerifierService.h

@@ -1,24 +1,24 @@
 // -*- tab-width: 2; indent-tabs-mode: nil; coding: utf-8-with-signature -*-
 //-----------------------------------------------------------------------------
-// Copyright 2000-2022 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
+// Copyright 2000-2024 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
 // See the top-level COPYRIGHT file for details.
 // SPDX-License-Identifier: Apache-2.0
 //-----------------------------------------------------------------------------
 /*---------------------------------------------------------------------------*/
-/* VerifierService.h                                           (C) 2000-2022 */
+/* VerifierService.h                                           (C) 2000-2024 */
 /*                                                                           */
 /* Classe de base du service de vérification des variables.                  */
 /*---------------------------------------------------------------------------*/
-#ifndef ARCANE_VERIFIERSERVICE_H
-#define ARCANE_VERIFIERSERVICE_H
+#ifndef ARCANE_CORE_VERIFIERSERVICE_H
+#define ARCANE_CORE_VERIFIERSERVICE_H
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
 #include "arcane/utils/String.h"
 
-#include "arcane/ArcaneTypes.h"
-#include "arcane/AbstractService.h"
-#include "arcane/IVerifierService.h"
+#include "arcane/core/ArcaneTypes.h"
+#include "arcane/core/AbstractService.h"
+#include "arcane/core/IVerifierService.h"
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
@@ -44,26 +44,33 @@ class ARCANE_CORE_EXPORT VerifierService
 , public IVerifierService
 {
  private:
+
   struct DiffInfo
   {
    public:
+
     DiffInfo()
-    : m_variable_name(), m_nb_diff(0)
-      {}
-    DiffInfo(const String& var_name,Int64 nb_diff)
-    : m_variable_name(var_name), m_nb_diff(nb_diff)
-      {}
+    : m_variable_name()
+    , m_nb_diff(0)
+    {}
+    DiffInfo(const String& var_name, Int64 nb_diff)
+    : m_variable_name(var_name)
+    , m_nb_diff(nb_diff)
+    {}
+
    public:
+
     String m_variable_name;
     Int64 m_nb_diff;
   };
+
  public:
 
   explicit VerifierService(const ServiceBuildInfo& sbi);
   ~VerifierService() override;
 
  public:
-  
+
   IBase* serviceParent() const override;
   IServiceInfo* serviceInfo() const override { return m_service_info; }
   IService* serviceInterface() override { return this; }
@@ -78,35 +85,37 @@ class ARCANE_CORE_EXPORT VerifierService
   void setResultFileName(const String& file_name) override { m_result_file_name = file_name; }
   String resultfileName() const override { return m_result_file_name; }
 
- public:
-  
+  //! Type de comparaison souhaité
+  void setCompareMode(eCompareMode v) override { m_compare_mode = v; }
+  eCompareMode compareMode() const override { return m_compare_mode; }
+
   void setSubDir(const String& sub_dir) override { m_sub_dir = sub_dir; }
   String subDir() const override { return m_sub_dir; }
 
  protected:
 
-  virtual void _doVerif(IDataReader* reader,const VariableCollection& variables,bool compare_ghost);
+  virtual void _doVerif(IDataReader* reader, const VariableCollection& variables, bool compare_ghost);
   //! Remplit dans \a variables la liste des variables devant être relues
-  virtual void _getVariables(VariableList variables,bool parallel_sequential);
+  virtual void _getVariables(VariableList variables, bool parallel_sequential);
 
  public:
 
   ISubDomain* subDomain() { return m_sub_domain; }
 
  protected:
-
  private:
 
-  ISubDomain* m_sub_domain;
-  IServiceInfo* m_service_info;
+  ISubDomain* m_sub_domain = nullptr;
+  IServiceInfo* m_service_info = nullptr;
   String m_file_name;
   String m_result_file_name;
   String m_sub_dir;
+  eCompareMode m_compare_mode = eCompareMode::Values;
 
  private:
 
-  template<typename ReaderType> void
-  _doVerif2(ReaderType reader,const VariableList& variables,bool compare_ghost);
+  template <typename ReaderType> void
+  _doVerif2(ReaderType reader, const VariableList& variables, bool compare_ghost);
 };
 
 /*---------------------------------------------------------------------------*/