Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LIMA refactoring and cleaning #57

Open
1 of 19 tasks
SebastienRietteMTO opened this issue Oct 11, 2024 · 0 comments
Open
1 of 19 tasks

LIMA refactoring and cleaning #57

SebastienRietteMTO opened this issue Oct 11, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@SebastienRietteMTO
Copy link
Collaborator

SebastienRietteMTO commented Oct 11, 2024

Several modifications have been identified:

  1. Setup:

    • setup doesn't work in simple precision (eg on NEC: "ABOR1 [PROC=1,THRD=1] : PA argument is too large or ITMAX is too small, the incomplete GAMMA_INC function cannot be evaluated correctly by the continuous fraction method")
    • setup doesn't work with NVFORTRAN (deactivated in lima and lima_adjust testprogs with OPENACC)
  2. lima and lima_adjust common issues

    • IN/OUT/INOUT
    • use 2 dimensions instead of 3
    • use structures (instead of using values from MODD_NSV, MODD_PARAM_LIMA… for lima; NEBn, TURBn… to be passed through the interface for lima_adjust)
    • add DrHook
    • use explicit shapes in interfaces
    • conformity to Doctor norm (especially P for dummy args, Z for local variables)
    • code must be in upper cases
    • would it be possible to use interp_micro.func.h instead of ZVEC/IVEC?
    • ALLOCATABLE arrays must be removed
    • reactivate lima and lima_adjust calls with OPENACC in the testprogs
  3. issues specific to lima

    • PEVAP must be initialised with 0
    • useless arguments NIFN, NCCN, NIMM must be suppressed
  4. issues specific to lima_adjust

    • CFw is not reset to 0 after the control « remove cloud droplets if there are few ». Would it be a solution to call lima_compute_fractions here?
    • code still uses 3 time steps whereas the third time step has been removed from other parametrisations
    • there is an asymmetry between C et I (PCIT/PCIS vs PCCT/PCCS), is it OK?
    • NBUMOD and KMI must be suppressed
    • PSIGQSAT must be a 2D array
@SebastienRietteMTO SebastienRietteMTO added enhancement New feature or request good first issue Good for newcomers labels Oct 11, 2024
@SebastienRietteMTO SebastienRietteMTO self-assigned this Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant