Opened 7 years ago

Last modified 9 months ago

#245 accepted defect

Clarify the coding and usage of frac_nobio

Reported by: jpolcher Owned by: maignan
Priority: major Milestone: Work on sft and multitiling
Component: Physical processes Version: trunc
Keywords: Cc:


An analysis of the usage of frac_nobio in the model ( has revealed some issues which need to be addressed. This will clarify the code and allow new evolutions which are planed.

frac_nobio and totfrac_nobio contain redundant information as totfrac_nobio=SUM(frac_nobio) over nnobio. But both are passed to most subroutines and nowhere is it verified that the information matches.

The code would be much more easier to follow if only frac_nobio would be passed. When totfrac_nobio is needed the sum would be done locally.

totfrac_nobio is for instance the only variable used for STOMATE and this is logical. So at least slowproc.f90 will need to compute this variable to pass it to STOMATE.

The other issue is that there is no guarantee that the model conserves all stocks if frac_nobio changes over time. So the model should check that frac_nobio does not change. Especially in slowproc.f90 where frac_nobio can change with the reading and interpolation of the PFT maps.

Furthermore the introduction of nnobio has not been done fully. Many other variables need to have this dimension as well, for instance the parameters to be used on the surfaces other than ice.

Change History (5)

comment:1 Changed 6 years ago by jgipsl

  • Owner changed from somebody to maignan
  • Status changed from new to assigned

comment:2 Changed 6 years ago by maignan

  • Component changed from Anthropogenic processes to Physical processes
  • Milestone set to ORCHIDEE 3.0
  • Status changed from assigned to accepted
  • Version set to trunc

comment:3 Changed 5 years ago by maignan

  • Milestone changed from ORCHIDEE 3.0 to ORCHIDEE 4.0

comment:4 Changed 3 years ago by luyssaert

  • Milestone changed from ORCHIDEE 4.0 to Not scheduled yet

comment:5 Changed 9 months ago by jgipsl

  • Milestone changed from Not scheduled yet to sft_and_multitiling
Note: See TracTickets for help on using tickets.