Skip to content

Commit

Permalink
Fix bug from issue crest-lab#296
Browse files Browse the repository at this point in the history
Signed-off-by: Philipp Pracht <pp555@cam.ac.uk>
  • Loading branch information
pprcht committed May 3, 2024
1 parent 10fc64c commit 3a37ffe
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
5 changes: 3 additions & 2 deletions src/algos/refine.f90
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ subroutine crest_refine(env,input,output)
character(len=*),intent(in) :: input
character(len=*),intent(in),optional :: output
!===========================================================!
integer :: i,j,k,l,io,ich,m
integer :: i,j,k,l,io,ich,m,t1,t2
logical :: pr,wr,ex
!===========================================================!
character(len=:),allocatable :: outname
Expand Down Expand Up @@ -98,8 +98,9 @@ subroutine crest_refine(env,input,output)
call crest_oloop(env,nat,nall,at,xyz,eread,.false.)

case(refine%confsolv)
call new_ompautoset(env,'subprocess',1,t1,t2)
write (stdout,'("> ConfSolv: ΔΔGsoln estimation from 3D directed message passing neural networks (D-MPNN)")')
call confsolv_request( input, nall, env%threads, etmp, io)
call confsolv_request( input, nall, t2, etmp, io)
if(io == 0)then
eread(:) = etmp(:)*kcaltoau !> since CREGEN deals with Eh energies
endif
Expand Down
2 changes: 1 addition & 1 deletion src/axis_module.f90
Original file line number Diff line number Diff line change
Expand Up @@ -336,7 +336,7 @@ subroutine axistrf(nat,nat0,at,coord)
allocate (attmp(nat0))
allocate (coordtmp(3,nat0))
attmp(1:nat0) = at(1:nat0)
coordtmp(3,1:nat0) = coord(3,1:nat0)
coordtmp(1:3,1:nat0) = coord(1:3,1:nat0)
call axis_0(nat0,attmp,coordtmp,rot,avmom,evec)
deallocate (coordtmp,attmp)

Expand Down
9 changes: 7 additions & 2 deletions src/qcg/solvtool_misc.f90
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ subroutine xtb_lmo(env, fname)!,chrg)
character(len=*), intent(in) :: fname
character(len=80) :: pipe
character(len=512) :: jobcall
integer :: T,Tn
integer :: T,Tn,io

pipe = ' > xtb.out 2>/dev/null'

Expand All @@ -125,7 +125,12 @@ subroutine xtb_lmo(env, fname)!,chrg)
!---- jobcall, special gbsa treatment not needed, as the entire flag is included in env%solv
write (jobcall, '(a,1x,a,1x,a,'' --sp --lmo '',a)') &
& trim(env%ProgName), trim(fname), trim(env%lmover), trim(pipe)
call command(trim(jobcall))
call command(trim(jobcall), exitstat=io)

if(io /= 0)then
write(*,*) 'error in xtb_lmo'
stop
endif

!--- cleanup
call remove('wbo')
Expand Down

0 comments on commit 3a37ffe

Please sign in to comment.