-
Couldn't load subscription status.
- Fork 8
560 new calculation for accounting of root distribution on the transpiration #591
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
base: main
Are you sure you want to change the base?
Changes from all commits
6a04a44
3334097
4eacb1c
fcce19c
332b272
defcb20
f60d767
7bd21f4
43c8ffe
4d0093f
4c1c924
5520024
f097c1e
e752a90
71447e5
f6b58b3
24ef042
7e83ba7
84ed819
aa9c084
d6acbe3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1930,7 +1930,7 @@ SUBROUTINE master_cable_params (comm,met,air,ssnow,veg,bgc,soil,canopy,& | |
|
|
||
| bidx = bidx + 1 | ||
| CALL MPI_Get_address (ssnow%evapfbl(off,1), displs(bidx), ierr) | ||
| CALL MPI_Type_create_hvector (ms, r1len, r1stride, MPI_BYTE, & | ||
| CALL MPI_Type_create_hvector (ms, r2len, r2stride, MPI_BYTE, & | ||
| & types(bidx), ierr) | ||
| blen(bidx) = 1 | ||
|
|
||
|
|
@@ -2508,14 +2508,6 @@ SUBROUTINE master_cable_params (comm,met,air,ssnow,veg,bgc,soil,canopy,& | |
| ! blen(bidx) = 1 | ||
| ! !blen(bidx) = ms * r1len | ||
|
|
||
| bidx = bidx + 1 | ||
| CALL MPI_Get_address (canopy%evapfbl(off,1), displs(bidx), ierr) | ||
| ! MPI: gol124: changed to r1 when Bernard ported to CABLE_r491 | ||
| CALL MPI_Type_create_hvector (ms, r1len, r1stride, MPI_BYTE, & | ||
| & types(bidx), ierr) | ||
| blen(bidx) = 1 | ||
| !blen(bidx) = ms * r2len | ||
|
|
||
| bidx = bidx + 1 | ||
| CALL MPI_Get_address (canopy%epot(off), displs(bidx), ierr) | ||
| blen(bidx) = r1len | ||
|
|
@@ -4788,12 +4780,6 @@ SUBROUTINE master_outtypes (comm,met,canopy,ssnow,rad,bal,air,soil,veg) | |
| ! REAL(r_2) | ||
| ! MPI: gol124: backport to r1134 changes r_2 to r_1 | ||
| ! MPI: gol124: in newest CABLE-cnp it's r_2 again | ||
| midx = midx + 1 | ||
| CALL MPI_Get_address (canopy%evapfbl(off,1), maddr(midx), ierr) ! 2 | ||
| ! MPI: gol124: changed to r1 when Bernard ported to CABLE_r491 | ||
| CALL MPI_Type_create_hvector (ms, r1len, r1stride, MPI_BYTE, & | ||
| & mat_t(midx, rank), ierr) | ||
| CALL MPI_Type_commit (mat_t(midx, rank), ierr) | ||
|
|
||
| midx = midx + 1 | ||
| CALL MPI_Get_address (canopy%gswx(off,1), maddr(midx), ierr) ! 2 | ||
|
|
@@ -4876,9 +4862,9 @@ SUBROUTINE master_outtypes (comm,met,canopy,ssnow,rad,bal,air,soil,veg) | |
| & mat_t(midx, rank), ierr) | ||
| CALL MPI_Type_commit (mat_t(midx, rank), ierr) | ||
| midx = midx + 1 | ||
| ! REAL(r_1) | ||
| ! REAL(r_2) | ||
| CALL MPI_Get_address (ssnow%evapfbl(off,1), maddr(midx), ierr) ! 12 | ||
| CALL MPI_Type_create_hvector (ms, r1len, r1stride, MPI_BYTE, & | ||
| CALL MPI_Type_create_hvector (ms, r2len, r2stride, MPI_BYTE, & | ||
| & mat_t(midx, rank), ierr) | ||
| CALL MPI_Type_commit (mat_t(midx, rank), ierr) | ||
| midx = midx + 1 | ||
|
|
@@ -7222,13 +7208,6 @@ SUBROUTINE master_restart_types (comm, canopy, air, bgc) | |
| ! & types(bidx), ierr) | ||
| ! blocks(bidx) = 1 | ||
|
|
||
| bidx = bidx + 1 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This change here is consistent with the intent of the PR but it highlights something missing (I think) Since you've changed the TYPE of There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just a check - does the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I believe the line 2513 you are referring to has been removed. It was for There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I can't find any reference to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
I think this must have been fine since the kind was not specified for There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @SeanBryan51 But would it be fine with the PR's changes given that bidx is pointing (not in the sense of a POINTER) to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @har917 , There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. My concern isn't really However - some of the variables are clearly being dealt with differently - see line 1983 for Perhaps what I'm really getting at here is that this probably needs a short MPI test of just the change in KIND on There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Yes I think so as long as
SeanBryan51 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| CALL MPI_Get_address (canopy%evapfbl(off,1), displs(bidx), ierr) ! 2 | ||
| ! MPI: gol124: changed to r1 when Bernard ported to CABLE_r491 | ||
| CALL MPI_Type_create_hvector (ms, r1len, r1stride, MPI_BYTE, & | ||
| & types(bidx), ierr) | ||
| blocks(bidx) = 1 | ||
|
|
||
| bidx = bidx + 1 | ||
| CALL MPI_Get_address (bgc%cplant(off,1), displs(bidx), ierr) | ||
| CALL MPI_Type_create_hvector (ncp, r1len, r1stride, MPI_BYTE, & | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.