commit de david

[hpcc2014.git] / hpcc2014_reviews.txt

diff --git a/hpcc2014_reviews.txt b/hpcc2014_reviews.txt
index 63a9279a147208e8549faf9a4044047621e59a88..2c5393aa8d4fe7c0ba4e842c844070201fd31af8 100644 (file)
--- a/hpcc2014_reviews.txt
+++ b/hpcc2014_reviews.txt
@@ -106,10 +106,10 @@ AUTHORS: Charles Emile Ramamonjisoa, David Laiymani, Arnaud Giersch,
       En quelque sorte, on a bien modifié l'algorithme de GMRES pour l'adapter
       aux clusters distants. On a utilisé des itérations asynchrones pour
       recouvrir les communications par du calcul et le multisplittig pour
-      réduire le volume total des communications. De toute façon, on ne peut
+      réduire le volume total des communications. De toute façon, on ne pouvait
       pas appliquer les itérations asynchrones sur GMRES sans le multisplitting.
-      On peut bien sûr utiliser ces deux techniques sur d'autres méthodes
-      numériques de résolution.  
+      On peut bien sûr utiliser ces deux techniques avec une autre méthode
+      numérique de résolution comme solveur interne.  
 
 ,----
 | The prejudice of the paper is that the GMRES algorithm is not using
@@ -164,6 +164,7 @@ AUTHORS: Charles Emile Ramamonjisoa, David Laiymani, Arnaud Giersch,
 `----
 
 [RCE] Je pense que le reviewer a raison. Lilia ?
+[LZK] OUI, k==MaxIter.                                            
 
 ,----
 | As far as the reviewer can tell, SMPI removes heavy computation by
@@ -200,18 +201,18 @@ AUTHORS: Charles Emile Ramamonjisoa, David Laiymani, Arnaud Giersch,
 
 ,----
 | Some knitpicks include:
-| - Abstract: "Behaviours", please no plural
-| - Sec II (and others): "As exposed" --> "As described"
-| - Sec II: "And important idle times" --> better "useless idle times
-|   used for synchronization"
-| - Sec III: "by the mean of an XML file" --> "by means of an XML file".
-| - SEC IV.B: did not encouter ... unless some code debugging" -->
-|   please rewrite the unless part...
-| - SEC V: "Hosts processors power" --> "Host processor power"
+| [x] Abstract: "Behaviours", please no plural
+| [x] Sec II (and others): "As exposed" --> "As described"
+| [ ] Sec II: "And important idle times" --> better "useless idle times
+|     used for synchronization"
+| [x] Sec III: "by the mean of an XML file" --> "by means of an XML file".
+| [ ] SEC IV.B: did not encouter ... unless some code debugging" -->
+|     please rewrite the unless part...
+| [x] SEC V: "Hosts processors power" --> "Host processor power"
 `----
 
 [RCE] On va prendre en compte ces remarques.
- [AG] J'ai commencé pour les plus faciles.
+ [AG] J'ai commencé pour les plus faciles (cochées dans la liste ci-dessus).
 
 
 ----------------------- REVIEW 3 ---------------------
@@ -255,6 +256,9 @@ AUTHORS: Charles Emile Ramamonjisoa, David Laiymani, Arnaud Giersch,
 `----
 
 [RCE] ??
+[RC] En gros il nous critique car on a pris un problème trop simple.
+                On le sait, mais on fait de l'asynchrone. Donc on ne répondra pas à 
+                cette remarque. 
 
 ,----
 | 3) This is somewhat of a minor point, but I did not see an explicit
@@ -271,6 +275,16 @@ AUTHORS: Charles Emile Ramamonjisoa, David Laiymani, Arnaud Giersch,
       IV. Il permet effectivement de determiner le critère ou la
       condition de convergence globale. Lilia peut confirmer ?
 
+[LZK] En asynchrone, on a toujours utilisé cette condition pour détecter
+      la convergence. Pour la convergence du solveur GMRES sur chaque
+      cluster (indépendemment de la convergence globale) on utilise ce
+      critère: || A x - b|| / || b ||<=inner precision. Pour la convergence 
+      globale, elle est détectée quand tous les calculs locaux ont 
+      convergé: (k==MaxIter)or(X^k−X^k+1)<=epsilon, X sous-vecteur
+      local de la solution et epsilon est la outer precision et donc
+      la précision donnée dans la table I. 
+[RC]   J'ai ajouté un truc là dessus, section IV A
+
 ,----
 | 4) Typical latencies within clusters are on the order of a
 | microsecond, and the latency used to produce Table I is more than