Previous Thread
Next Thread
Print Thread
Joined: Aug 2014
Posts: 2
A
andersx Offline OP
Forum Member
OP Offline
Forum Member
A
Joined: Aug 2014
Posts: 2
Hi Charmm

Why is the QChem module being enabled by default in c41a1?
It breaks certain QM/MM options with the DFTB module.

E.g. Do a simple compilation to enable the DFTB module like:

./install.com gnu T gfortran +DFTBMKL

and you cannot use the REMOVE option (Classical energies within QM atoms are removed) in the SCCDFTB group. The offending check is in nbonds/nbonds.src line 316, and should never affect the DFTB module from CHARMM.

It is of course pretty easy to fix by editing install.com and removing qchem from the default option (this in turn messes up the if clauses in mndo97_ltm.src, however, but also trivial to fix).
So there any reason the QChem module is enabled by default as I think it is not at all necessary for most users anyway?

Thanks
Anders Christensen


Last edited by andersx; 11/05/15 11:41 PM.
Joined: Sep 2003
Posts: 8,605
Likes: 23
rmv Offline
Forum Member
Offline
Forum Member
Joined: Sep 2003
Posts: 8,605
Likes: 23
A number of sites use the Q-Chem interface, so it's likely one of them submitted that change. Have you considered submitting your fixes for code coexistence so that c41a2 might be fixed?


Rick Venable
computational chemist

Joined: Sep 2003
Posts: 4,849
Likes: 6
Forum Member
Offline
Forum Member
Joined: Sep 2003
Posts: 4,849
Likes: 6
This is part of the effort to allow runtime selection of which QM code to use.


Lennart Nilsson
Karolinska Institutet
Stockholm, Sweden
Joined: Aug 2014
Posts: 2
A
andersx Offline OP
Forum Member
OP Offline
Forum Member
A
Joined: Aug 2014
Posts: 2
Thanks for the speedy replies! In that case I'd be very hesitant to push any bugfixes, as I don't know the reasoning behind the specific changes (other than runtime selection of QM code.) Any changes I make might just in turn break (or disable) the QChem module.

I put an example that is broken here:
https://dl.dropboxusercontent.com/u/17435887/bugreport.tar.bz2

The examples needs updated filepaths, runs with previous versions of CHARMM, but is broken in c41a1.


Moderated by  BRBrooks, bucknerj, lennart, rmv 

Link Copied to Clipboard
Powered by UBB.threads™ PHP Forum Software 7.7.5
(Release build 20201027)
Responsive Width:

PHP: 7.3.31-1~deb10u1 Page Time: 0.008s Queries: 22 (0.004s) Memory: 0.7416 MB (Peak: 0.7908 MB) Data Comp: Off Server Time: 2022-09-26 12:27:22 UTC
Valid HTML 5 and Valid CSS