Branche Amaury dev - Historique des versions

Abarral le 23 septembre 2024 à 08:14

2024-09-23T08:14:16Z

Abarral le 9 septembre 2024 à 13:40

2024-09-09T13:40:32Z

Abarral le 3 août 2024 à 07:37

2024-08-03T07:37:33Z

Abarral le 2 août 2024 à 17:16

2024-08-02T17:16:33Z

Abarral le 2 août 2024 à 15:49

2024-08-02T15:49:47Z

Abarral le 2 août 2024 à 13:03

2024-08-02T13:03:41Z

Abarral le 2 août 2024 à 12:54

2024-08-02T12:54:17Z

Abarral le 2 août 2024 à 11:17

2024-08-02T11:17:51Z

Abarral le 2 août 2024 à 11:17

2024-08-02T11:17:36Z

Abarral le 2 août 2024 à 10:50

2024-08-02T10:50:07Z

@@ Ligne 40 : / Ligne 40 : @@
   All good ! for r5157/r5017=-cosp v1 -rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
-(XIOS n'est pas testé vs 5150 car il segfault sur 5017)
+(XIOS est testé vs 5150 car il segfault sur 5017)
 == Opérations mises "de côté" ==

@@ Ligne 12 : / Ligne 12 : @@
 * La mise à jour des sources FCM à la dernière version (mais sans passer à FCM2, on utilise encore les options legacy)
 * La correction de bugs mineurs (e.g. type casting) et majeurs (e.g. mauvais indices de boucles, appels erronés) trouvés lors du nettoyage
-* Le renommage des modules en <code>ldmz_*</code> pour éviter les collisions de namespace
+* Le renommage des modules en <code>lmdz_*</code> pour éviter les collisions de namespace
 * L'harmonisation de la syntaxe Fortran (cf [[HowTo:_LMDZ_code_guidelines | LMDZ_code_guidelines]])
 * Un linting de la plupart des fichiers

← Version précédente		Version du 3 août 2024 à 07:37
Ligne 86 :		Ligne 86 :
	* compil lmdz: <code>Time (mean ± σ): 130.779 s ± 5.543 s \| Range (min … max): 123.503 s … 139.438 s 10 runs</code>		* compil lmdz: <code>Time (mean ± σ): 130.779 s ± 5.543 s \| Range (min … max): 123.503 s … 139.438 s 10 runs</code>
	* bench: <code>Time (mean ± σ): 52.444 s ± 1.080 s \| Range (min … max): 51.387 s … 54.530 s 10 runs</code>		* bench: <code>Time (mean ± σ): 52.444 s ± 1.080 s \| Range (min … max): 51.387 s … 54.530 s 10 runs</code>
		+
		+	=== r5159 ===
		+
		+	This rev replaces all <code>INCLUDE "dimensions.h",INCLUDE"paramet.h"</code> by modules. One could think this would impact performance.
		+	The bench suggests otherwise:
		+
		+	Seq: <code>Time (mean ± σ): 101.333 s ± 20.405 s \| Range (min … max): 91.460 s … 150.911 s 10 runs</code>
		+
		+	Para: <code>Time (mean ± σ): 49.341 s ± 0.353 s \| Range (min … max): 48.740 s … 50.028 s 10 runs</code>

@@ Ligne 13 : / Ligne 13 : @@
 * La correction de bugs mineurs (e.g. type casting) et majeurs (e.g. mauvais indices de boucles, appels erronés) trouvés lors du nettoyage
 * Le renommage des modules en <code>ldmz_*</code> pour éviter les collisions de namespace
-* L'harmonisation de la syntaxe Fortran (cf [https://lmdz-forge.lmd.jussieu.fr/mediawiki/LMDZPedia/index.php/HowTo:_LMDZ_code_guidelines])
+* L'harmonisation de la syntaxe Fortran (cf [[HowTo:_LMDZ_code_guidelines | LMDZ_code_guidelines]])
 * Un linting de la plupart des fichiers
@@ Ligne 63 : / Ligne 63 : @@
 == Vitesse ==
-A titre indicatif, quelques stats. Attention à l'interprétation (variabilité interne, etc.).
+A titre indicatif, quelques stats. Attention à l'interprétation (variabilité interne, etc.), la quantité intéressant est le <code>min</code>.
 === r5158 v r5017 ===
@@ Ligne 80 : / Ligne 80 : @@
 r5017:
-* compil lmdz: 80s
+* compil lmdz: <code>Time (mean ± σ):     119.116 s ±  4.124 s | Range (min … max):   114.514 s … 125.466 s    10 runs</code>
-* bench: 55s
+* bench: <code>Time (mean ± σ):     50.382 s ±  0.994 s | Range (min … max):   49.248 s … 52.130 s    10 runs</code>
 r5158
-* compil lmdz: 126s
+* compil lmdz:  <code>Time (mean ± σ):     130.779 s ±  5.543 s | Range (min … max):   123.503 s … 139.438 s    10 runs</code>
-* bench: 46-s
+* bench: <code>Time (mean ± σ):     52.444 s ±  1.080 s | Range (min … max):   51.387 s … 54.530 s    10 runs</code>

@@ Ligne 70 : / Ligne 70 : @@
 r5017:
-* compil lmdz: 100s
+* compil lmdz: <code>Time (mean ± σ):     152.594 s ± 62.315 s | Range (min … max):   109.719 s … 263.752 s    10 runs</code>
-* bench: 86s
+* bench: <code>Time (mean ± σ):     123.119 s ± 42.065 s | Range (min … max):   89.570 s … 200.606 s    10 runs</code>
 r5158
-* compil lmdz: 115s
+* compil lmdz: <code>Time (mean ± σ):     111.737 s ±  4.910 s | Range (min … max):   106.716 s … 118.421 s    10 runs</code>
-* bench: 100s
+* bench: <code>Time (mean ± σ):     92.876 s ±  2.105 s | Range (min … max):   87.766 s … 94.780 s    10 runs</code>
 ==== mpi_omp ====
@@ Ligne 85 : / Ligne 85 : @@
 r5158
 * compil lmdz: 126s
-* bench: 47s
+* bench: 46-s

← Version précédente		Version du 2 août 2024 à 13:03
Ligne 76 :		Ligne 76 :
	* compil lmdz: 115s		* compil lmdz: 115s
	* bench: 100s		* bench: 100s
		+
		+	==== mpi_omp ====
		+
		+	r5017:
		+	* compil lmdz: 80s
		+	* bench: 55s
		+
		+	r5158
		+	* compil lmdz: 126s
		+	* bench: 47s

← Version précédente		Version du 2 août 2024 à 12:54
Ligne 60 :		Ligne 60 :
	Le common défini dans <code>ini_COSP.h</code> est un peu tricky à remplacer car il a besoin de <code>klon,klev</code> qui ne sont pas des constantes.		Le common défini dans <code>ini_COSP.h</code> est un peu tricky à remplacer car il a besoin de <code>klon,klev</code> qui ne sont pas des constantes.
	Une (bonne) manière de faire serait de faire un type dérivé. C'est d'ailleurs ce que je préconise pour remplacer les très longues listes d'arguments, mais c'est à discuter avec le reste du groupe...		Une (bonne) manière de faire serait de faire un type dérivé. C'est d'ailleurs ce que je préconise pour remplacer les très longues listes d'arguments, mais c'est à discuter avec le reste du groupe...
		+
		+	== Vitesse ==
		+
		+	A titre indicatif, quelques stats. Attention à l'interprétation (variabilité interne, etc.).
		+
		+	=== r5158 v r5017 ===
		+
		+	==== séquentiel ====
		+
		+	r5017:
		+	* compil lmdz: 100s
		+	* bench: 86s
		+
		+	r5158
		+	* compil lmdz: 115s
		+	* bench: 100s

← Version précédente		Version du 2 août 2024 à 11:17
Ligne 59 :		Ligne 59 :

	Le common défini dans <code>ini_COSP.h</code> est un peu tricky à remplacer car il a besoin de <code>klon,klev</code> qui ne sont pas des constantes.		Le common défini dans <code>ini_COSP.h</code> est un peu tricky à remplacer car il a besoin de <code>klon,klev</code> qui ne sont pas des constantes.
−	Une (bonne)à manière de faire serait de faire un type dérivé. C'est d'ailleurs ce que je préconise pour remplacer les très longues listes d'arguments, mais c'est à discuter avec le reste du groupe...	+	Une (bonne) manière de faire serait de faire un type dérivé. C'est d'ailleurs ce que je préconise pour remplacer les très longues listes d'arguments, mais c'est à discuter avec le reste du groupe...

@@ Ligne 52 : / Ligne 52 : @@
 === Remplacement du commun comgeom ===
-Le commun <code>/comgeom/</code> est partagé entre <code>lmdz_comgeom.f90</code> et <code>lmdz_comgeom2.f90</code>, afin de pouvoir de manière transparents passer les même tableaux en tant que 1D ou 2D.
+Le common <code>/comgeom/</code> est partagé entre <code>lmdz_comgeom.f90</code> et <code>lmdz_comgeom2.f90</code>, afin de pouvoir de manière transparents passer les même tableaux en tant que 1D ou 2D.
 Cette utilisation est dépréciée par Fortran, mais pas triviale à changer en module.
 Une solution serait de passer par des pointeurs et une routine d'initialisation.

@@ Ligne 21 : / Ligne 21 : @@
    # Tests auto de convergence
-  All good ! for r5017/r5155=-rad oldrad -parallel none -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad oldrad -parallel none -veget none -compilephysiq lmd
-  All good ! for r5017/r5155=-rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
-  All good ! for r5017/r5155=-rad oldrad -parallel none -veget CMIP6 -compilephysiq lmd
+ All good ! for r5155/r5017=-rad oldrad -parallel none -veget CMIP6 -compilephysiq lmd
-  All good ! for r5017/r5155=-rad oldrad -parallel mpi_omp -veget CMIP6 -compilephysiq lmd
+ All good ! for r5155/r5017=-rad oldrad -parallel mpi_omp -veget CMIP6 -compilephysiq lmd
-  All good ! for r5017/r5155=-rad rrtm -parallel none -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad rrtm -parallel none -veget none -compilephysiq lmd
-  All good ! for r5017/r5155=-rad rrtm -parallel mpi_omp -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad rrtm -parallel mpi_omp -veget none -compilephysiq lmd
-  All good ! for r5017/r5155=-rad rrtm -parallel none -veget CMIP6 -compilephysiq lmd
+ All good ! for r5155/r5017=-rad rrtm -parallel none -veget CMIP6 -compilephysiq lmd
-  All good ! for r5017/r5155=-rad rrtm -parallel mpi_omp -veget CMIP6 -compilephysiq lmd
+ All good ! for r5155/r5017=-rad rrtm -parallel mpi_omp -veget CMIP6 -compilephysiq lmd
-  All good ! for r5017/r5155=-rad ecrad -parallel none -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad ecrad -parallel none -veget none -compilephysiq lmd
-  All good ! for r5017/r5155=-rad ecrad -parallel mpi_omp -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad ecrad -parallel mpi_omp -veget none -compilephysiq lmd
-  All good ! for r5017/r5155=-rad ecrad -parallel none -veget CMIP6 -compilephysiq lmd
+ All good ! for r5155/r5017=-rad ecrad -parallel none -veget CMIP6 -compilephysiq lmd
-  All good ! for r5017/r5155=-rad ecrad -parallel mpi_omp -veget CMIP6 -compilephysiq lmd
+ All good ! for r5155/r5017=-rad ecrad -parallel mpi_omp -veget CMIP6 -compilephysiq lmd
-  All good ! for r5017/r5155=-debug -rad oldrad -parallel none -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-debug -rad oldrad -parallel none -veget none -compilephysiq lmd
-  All good ! for r5017/r5157=-cosp v1 -rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-debug -rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
+ All good ! for r5155/r5017=-rad oldrad -parallel none -veget none -compilephysiq lmd -d 79
+ All good ! for r5155/r5017=-rad oldrad -parallel none -veget none -compilephysiq lmdiso -d 48x36x39
-XIOS n'est pas testé vs 5017, car il segfault sur trunk à cette révision, mais on a bien convergence avec r5150:
+ All good ! for r5157/r5150=-xios -rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
-  All good ! for r5150/r5157=-xios -rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
+ All good ! for r5157/r5017=-cosp v1 -rad oldrad -parallel mpi_omp -veget none -compilephysiq lmd
 == Opérations mises "de côté" ==