Skip to content

Commit

Permalink
Random U and N where sharing the same state object. Make these states…
Browse files Browse the repository at this point in the history
… unqiue and remove nested destructor call this states are in a global context.
  • Loading branch information
cianciosa committed Mar 6, 2024
1 parent 5c4aefe commit ee11ffb
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 10 deletions.
20 changes: 12 additions & 8 deletions src/korc_random.f90
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@ MODULE korc_random
IMPLICIT NONE

TYPE(C_PTR), DIMENSION(:), ALLOCATABLE , PRIVATE :: states
TYPE(C_PTR), PRIVATE :: state
TYPE(C_PTR), PRIVATE :: state_u
TYPE(C_PTR), PRIVATE :: state_n
! TYPE(VSL_STREAM_STATE), PRIVATE :: stream

INTERFACE
Expand Down Expand Up @@ -130,40 +131,43 @@ SUBROUTINE finalize_random
IMPLICIT NONE

INTEGER :: thread_num
!$OMP PARALLEL

!$OMP PARALLEL PRIVATE(thread_num)
thread_num = get_thread_number()
CALL random_destroy_U(states(thread_num))
!$OMP END PARALLEL

DEALLOCATE (states)
END SUBROUTINE

SUBROUTINE initialize_random_U(seed)
IMPLICIT NONE

INTEGER, INTENT(IN) :: seed

state = random_construct_U(seed)
state_u = random_construct_U(seed)

END SUBROUTINE initialize_random_U

SUBROUTINE finalize_random_U
IMPLICIT NONE

CALL random_destroy_U(state)
CALL random_destroy_U(state_u)
END SUBROUTINE

SUBROUTINE initialize_random_N(seed)
IMPLICIT NONE

INTEGER, INTENT(IN) :: seed

state = random_construct_N(seed)
state_n = random_construct_N(seed)

END SUBROUTINE initialize_random_N

SUBROUTINE finalize_random_N
IMPLICIT NONE

CALL random_destroy_N(state)
CALL random_destroy_N(state_n)
END SUBROUTINE

FUNCTION get_random()
Expand All @@ -179,15 +183,15 @@ FUNCTION get_random_U()

REAL(rp) :: get_random_U

get_random_U = random_get_number_U(state)
get_random_U = random_get_number_U(state_u)
END FUNCTION get_random_U

FUNCTION get_random_N()
IMPLICIT NONE

REAL(rp) :: get_random_N

get_random_N = random_get_number_N(state)
get_random_N = random_get_number_N(state_n)
END FUNCTION get_random_N

SUBROUTINE get_randoms(nums)
Expand Down
3 changes: 1 addition & 2 deletions src/korc_velocity_distribution.f90
Original file line number Diff line number Diff line change
Expand Up @@ -415,6 +415,7 @@ subroutine gyro_distribution(params,F,spp)

call init_random_seed(params)
call RANDOM_NUMBER(theta)
call finalize_random_seed
theta = 2.0_rp*C_PI*theta

if (spp%spatial_distribution.eq.'TRACER') theta=2.0*C_PI
Expand Down Expand Up @@ -476,8 +477,6 @@ subroutine gyro_distribution(params,F,spp)
DEALLOCATE(b2)
DEALLOCATE(b3)
DEALLOCATE(hint)

call finalize_random_seed
end subroutine gyro_distribution


Expand Down

0 comments on commit ee11ffb

Please sign in to comment.