Changes between Version 21 and Version 22 of DevelopmentActivities/ORCHIDEE-MICT-IMBALANCE-P/Evaluation

Timestamp:

2015-12-07T10:57:38+01:00 (9 years ago)

Author:

maignan

Comment:

--

DevelopmentActivities/ORCHIDEE-MICT-IMBALANCE-P/Evaluation

@@ -1 @@
-= Evaluation =
-
-== Notes from M. Guimberteau ==
-
-=== 2015/08/27 ===
-
-* Bugs
-
-'''In stomate_wet_ch4_pt_ter_0.f90'''
+= Solved Issues =
+
+== Bug report in stomate_wet_ch4_pt_ter_0.f90 by M. Guimberteau on 2015/08/27 ==
 
 The following parameters need to be defined and this part of the code should be added:
@@ -23 +17 @@
 Albert: changed r2906
 
-=== 2015/08/24 ===
-
-* Bugs
-
-'''In thermosoil'''
+
+== Bug report in thermosoil by M. Guimberteau on 2015/08/24 ==
 
 The variable SoilTemp should be in the almaoutput condition of the loop and related to ptn_pftmean and not ptn:
@@ -66 +57 @@
 Albert: done in r2907.
 
-'''In intersurf and ioipslctrl'''
+== Bugs report in intersurf and ioipslctrl by M. Guimberteau on 2015/08/24 ==
 
 The variable ptn_snow_pftmean does not exist in the code anymore. The call histdef should be removed:
@@ -96 +87 @@
 
 
-----
-
-
-== Notes from F. Maignan ==
-
+== Bug report in thermosoil.f90 / thermosoil_finalize by F. Maignan on 2015/09/10 & 2015/08/31 ==
 I've started a 2 degree V6 evaluation at LSCE here:[[BR]]
 /home/surface3/maignan/ORCHIDEE/TESTS/test_MICT_V6/modipsl/modeles/ORCHIDEE/modipsl/config/ORCHIDEE_OL/OOL_SEC_STO_V6[[BR]]
 
-The modified code is in /home/surface3/maignan/ORCHIDEE/TESTS/test_MICT_V6/modipsl
-
-
-=== '''Bugs''' ===
-* '''thermosoil.f90 / thermosoil_finalize''' (2015/09/10 & 2015/08/31) 
+The modified code is in /home/surface3/maignan/ORCHIDEE/TESTS/test_MICT_V6/modipsl[[BR]]
+
 The gthick, gtemp and gpkappa variables were not saved in the restart file.[[BR]]
 The following correction should be applied:
@@ -145 +129 @@
 Albert: done in r2909.
 
-* '''stomate.f90 / call writerestart''' (2015/08/27)
+== Bug report in stomate.f90 / call writerestart by F. Maignan on 2015/08/27 ==
 The following correction should be applied:
 {{{
@@ -160 +144 @@
 Albert: done in r2910.
 
-* '''lpj_spitfire.f90 / spitfire''' (2015/08/27)
+== Bug report in lpj_spitfire.f90 / spitfire by F. Maignan on 2015/08/27 ==
 Add:
 {{{
@@ -171 +155 @@
 Albert: done in r2910.
 
-* '''stomate_permafrost_carbon.f90 / snow_interpol / ! 5. inter- or extrapolate''' (2015/08/24)[[BR]]
+== Bug report in stomate_permafrost_carbon.f90 / snow_interpol / ! 5. inter- or extrapolate by F. Maignan on 2015/08/24 ==
 {{{
 IF ( dzio(ip,iv) .GT. 0. ) THEN
@@ -183 +167 @@
 Albert: done in r2910.
 
-* '''stomate_Cforcing_name and stomate_forcing_name''' (2015/08/21)
+== Bug report in stomate_Cforcing_name and stomate_forcing_name by F. Maignan on 2015/08/21 ==
 I've found a problem with the variable Cforcing_name, which is declared both as a global save variable of the stomate module and as a save variable of the stomate_main procedure. It is read in stomate_initialize using a getin_p but indeed the value is not correctly transmitted to the stomate_main procedure. There is also a stomate_Cforcing_name in constantes_var.
 
@@ -194 +178 @@
 Albert: done in r2910.
 
+== Bug report on LIGHTNING_FILE by F. Maignan on 2015/08/24
+LIGHTNING_FILE should not be read if FIRE_DISABLE=y. [[BR]]
+Albert: done in r2911.
+
 === '''To be corrected''' ===
-* 2015/08/24
- - LIGHTNING_FILE should not be read if FIRE_DISABLE=y. Albert: done in r2911.
 * 2015/08/21
  - GRAZING_MAP should not be read if grassland management is not activated.
@@ -202 +188 @@
 
 
-== Tao ==
+== Bug report by T. Wang ==
 
 There is a small bug in our MICT version that might induce the failure of some point simulations. In subroutine enerbil_flux of enerbil.f90:
@@ -230 +216 @@
 Albert: done in r2912.
 
-== Chao ==
+== Bug report by C. Yue ==
 
 Please put these changes in the '''src_stomate/Grassland_Management.f90'''. They're used to protect the division-by-zero from happening. But I don't think this correction is related negative fuel problem. But it could lead to Nan value in fuel if you activate the GM (which is not your case as I checked the run.def you sent before).