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

[pull] master from crest-lab:master #32

Merged
merged 3 commits into from
Apr 26, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Empty file added include/.gitignore
Empty file.
6 changes: 5 additions & 1 deletion src/axis_module.f90
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,12 @@ module axis_module
module procedure CMAxyz
module procedure CMAv
end interface cma
public :: CMAtrf

contains
!========================================================================================!
!========================================================================================!
contains !> MODULE PROCEDURES START HERE
!========================================================================================!
!========================================================================================!
!> subroutine axis_0
!> This is the original axis routine for calculating the
Expand Down
20 changes: 17 additions & 3 deletions src/confparse.f90
Original file line number Diff line number Diff line change
Expand Up @@ -710,6 +710,12 @@ subroutine parseflags(env,arg,nra)
if(argument.eq.'-ohess') env%crest_ohess=.true.
exit

case ('-hess','-numhess') !> Numerical hessian
env%preopt = .false.
env%crestver = crest_numhessian
env%legacy = .false.
exit

case ('-trialopt') !> test optimization with topocheck
env%preopt = .false.
env%crestver = crest_trialopt
Expand Down Expand Up @@ -2425,9 +2431,17 @@ subroutine inputcoords(env,arg)
!>-- after this point there should always be an coord file present
if (.not.allocated(env%inputcoords)) env%inputcoords = 'coord'
call mol%open('coord')
!>-- shift to CMA and align according to rot.const.
if (env%crestver /= crest_solv .and. env%crestver /= crest_sp &
& .and. env%crestver /= crest_optimize) call axis(mol%nat,mol%at,mol%xyz)
!>-- shift to CMA and/or align according to rot.const. We have to be careful about this.
if (any((/ crest_sp, crest_optimize, crest_numhessian, crest_trialopt /) == env%crestver))then
!> some runtypes should only do a CMA translation, but no rotation
call CMAtrf(mol%nat,mol%nat,mol%at,mol%xyz)
else if (env%crestver == crest_solv)then
!> runtypes like qcg must not modify input coordinates!
continue
else
!> all other can align with rot. axis
call axis(mol%nat,mol%at,mol%xyz)
endif
!>-- overwrite coord
call mol%write('coord')

Expand Down
5 changes: 5 additions & 0 deletions src/legacy_algos/protonate.f90
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,11 @@ end subroutine xtblmo
call protclean
call prothead

!>--- check method
if(trim(env%gfnver) .eq. '--gff')then
error stop 'protonate unavailable with GFN-FF -> need LMOs'
endif

if (.not.allocated(env%ptb%atmap)) allocate (env%ptb%atmap(env%nat))
if (.not.env%ptb%strictPDT.and..not.env%ptb%fixPDT) then
!--- sort the input file (H atoms to the bottom)
Expand Down
5 changes: 5 additions & 0 deletions src/legacy_algos/tautomerize.f90
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,11 @@ subroutine tautomerize(env,tim)
call tautclean
call tauthead

!>--- check method
if(trim(env%gfnver) .eq. '--gff')then
error stop 'tautomerize unavailable with GFN-FF -> need LMOs'
endif

if (.not.allocated(env%ptb%atmap)) allocate (env%ptb%atmap(env%nat))
if (.not.env%ptb%strictPDT.and..not.env%ptb%fixPDT) then
!--- sort the input file (H atoms to the bottom)
Expand Down
Loading