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

VdW table too large ERROR #14

Closed
noeliaferruz opened this issue Apr 12, 2016 · 15 comments
Closed

VdW table too large ERROR #14

noeliaferruz opened this issue Apr 12, 2016 · 15 comments
Assignees

Comments

@noeliaferruz
Copy link

Hi guys,

I have a system with two ligands. 1 of them parameterized with cgenff latest version through command line the other with parameterize. It's basically running a simulation of pdb file 4MQT.
I run this system with both par/par_all36_prot.prm and par/par_all36_prot_mod.prm.
It turns that the first works straight away the second cannot find atomtype xxxx. which is found in the cgenff ligand.

Let me know if you need all the files.

best
N

@noeliaferruz
Copy link
Author

Let me find more specifically what's exactly promoting the error.

@noeliaferruz
Copy link
Author

It turns out that the clash is between the parameterize ligand and c36_mod.
The system which uses c36 runs in ACEMD.
The system which uses c36_mod produces this output: (let me know if you need files)

ERROR: file mapmdx.cpp line 751: VDW table too large
[...]
# - 
# SWAN device 0 is NVML device 2 with PCI-ID 0000:08:00.0 and serial [0321215002520] 
# NVML : 0 : PCI Width 16x/16x   Speed 3/3
# NVML : 0 : Fan 0%  Temp 27C    Mem Used 123MB Free 12164MB Total 12287MB
# NVML : 0 : Performance state 0
# NVML : 0 : Power 26.64W 
# NVML : 0 : Load 0 GPU 0 Mem 
# Configuration [Tesla K80] [370] [Linux]
#
#WARNING: warning: Conflict in redefined improper parameters for i crn1 nrn2 nrn2 nrc4
#WARNING: warning: Conflict in redefined improper parameters for i nrn2 hrm2 hrm2 crn1
#WARNING: warning: Conflict in redefined improper parameters for i crn1 nrn2 nrc4 nrn1
#WARNING: warning: Conflict in redefined improper parameters for i nrn1 cr33 crn1 hrm1
#WARNING: warning: Conflict in redefined improper parameters for i crn1 nrc4 nrn2 nrn1
#WARNING: warning: Conflict in redefined improper parameters for i nrn1 ct2 crn1 hrm1
# Topology reports 59246 atoms
# -  There are 3 long-range exceptions
# VDW table size 52

@noeliaferruz noeliaferruz changed the title cgenff - c36_mod clash parameterize - c36_mod clash Apr 12, 2016
@mj-harvey
Copy link
Contributor

Firstly, that should have terminated at the "VDW table too large" error. Did it continue past that point, or did you edit down the # of parameters and try again?
Secondly, you should see those 6 improper terms in the parameter. The error indicates that they are present multiple times, and with different values. The impropers don't get reparameterised, this must be a change between differing versions of ffs being used.
Could you show them to me, please?

@noeliaferruz
Copy link
Author

  1. I copied literally the output I got. I just removed the [...] part containing the license information.
  2. Right.

Attached files to generate the error and the outputs.
files.zip

@noeliaferruz
Copy link
Author

noeliaferruz commented Apr 17, 2016

To update, it seems the warnings aren't causing the error.
I just run another simulation with an exact setup and although the warnings were there the error didn't raise and the simulation run.

Also, it seems independent of the forcefield in this second case, as the old c36 also produced the VdW table too large error.

@noeliaferruz noeliaferruz changed the title parameterize - c36_mod clash VdW table too large ERROR Apr 17, 2016
@noeliaferruz
Copy link
Author

noeliaferruz commented Apr 17, 2016

It is actually a very reproducible error every time I have two large ligands, a membrane, a protein, water and ions. To me it seems something related to having too many atomtype definitions and the way acemd handles that.

I have tried all the possible parameterize-cgenff combinations for the two ligands and only removing one of the two (independently of which one) solves the problem. So it seems indeed something related to the amount of atom-types. It is a rather important issue I think, as it impedes running simulations with agonists and partial agonists at the same time.

@mj-harvey
Copy link
Contributor

Yes, charmm has lots of atom types, too many for a particular fixed size
data table within acemd. The good news is it should be easy to expand it.
Will try to fix it tomorrow
On 17 Apr 2016 20:04, "Noelia Ferruz" notifications@github.com wrote:

It is actually a very reproducible error every time I have two large
ligands, a membrane, a protein, water and ions. To me it seems something
related to having too many atomtype definitions and the way acemd handles
that.

I have tried all the possible parameterize-cgenff combinations for the two
ligands and only removing one of the two (independently of which one)
solves the problem. So it seems indeed something related to the amount of
atom-types. It is a rather important bug I think, as it impedes running
simulations with agonists and partial agonists at the same time.


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#14 (comment)

@noeliaferruz
Copy link
Author

Hi Matt,

Thanks for your reply. Did you fix this yesterday in the end?
If so, let me know and I might try to update acemd -if you tell me how-. This issue is currently hampering an important project.

Thanks,
Noelia

@mj-harvey
Copy link
Contributor

Hi Noelia

Turns out it's not a straightforward fix. The table is as large as the GPU
memory it inhabits permits. I'm working on some additional code changes to
accommodate the modification. I hope to have something for you tomorrow at
the latest. I will send you an acemd by email
On 19 Apr 2016 15:37, "Noelia Ferruz" notifications@github.com wrote:

Hi Matt,

Thanks for your reply. Did you fix this yesterday in the end?
If so, let me know and I might try to update acemd -if you tell me how-.
This issue is currently hampering an important project.

Thanks,
Noelia


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#14 (comment)

@noeliaferruz
Copy link
Author

Many thanks Matt.

@giadefa giadefa assigned giadefa and mj-harvey and unassigned giadefa Apr 21, 2016
@giadefa
Copy link
Contributor

giadefa commented May 10, 2016

Partially fixed in the next acemd version.

@giadefa giadefa closed this as completed May 10, 2016
@GerardBCN
Copy link

I have a similar situation. Simulations crash with the "VDW table too large" error when running in GPUGRID. But I ran a similar system not too long ago and worked perfectly. In that case we had 306 atomtypes and in the current simulations we have 299 atomtypes.... how does the VDW table get created?

@stefdoerr
Copy link
Contributor

Actually it only keeps the atom types that are in the system you simulate and these are hard-coded to 54. You have 78

In [2]: mol = Molecule('./Downloads/system.psf')

In [3]: mol.atomtype
Out[3]: array(['CT3', 'HA3', 'HA3', ..., 'CLA', 'CLA', 'CLA'], dtype=object)

In [4]: np.unique(mol.atomtype)
Out[4]: 
array(['C', 'CA', 'CAI', 'CC', 'CEL1', 'CG2O5', 'CG2R61', 'CG301', 'CG314',
       'CG321', 'CG331', 'CG334', 'CL', 'CLA', 'CP1', 'CP2', 'CP3', 'CPH1',
       'CPH2', 'CPT', 'CRL1', 'CRL2', 'CT1', 'CT2', 'CT2A', 'CT3', 'CTL1',
       'CTL2', 'CTL3', 'CTL5', 'CY', 'H', 'HA1', 'HA2', 'HA3', 'HAL1',
       'HAL2', 'HAL3', 'HB1', 'HB2', 'HC', 'HEL1', 'HGA1', 'HGA2', 'HGA3',
       'HGP2', 'HGR61', 'HL', 'HOL', 'HP', 'HR1', 'HR3', 'HS', 'HT', 'N',
       'NC2', 'NG3P1', 'NH1', 'NH2', 'NH3', 'NR1', 'NR2', 'NTL', 'NY', 'O',
       'O2L', 'OBL', 'OC', 'OG2D3', 'OH1', 'OHL', 'OSL', 'OSLP', 'OT',
       'PL', 'S', 'SM', 'SOD'], dtype=object)

In [5]: len(np.unique(mol.atomtype))
Out[5]: 78

@GerardBCN
Copy link

Just for the record, a recent similar system that I simulated had 82 atomtypes (when calculating it using the same protocol) so there must be something else wrong...

@stefdoerr
Copy link
Contributor

My bad, it also joins atomtypes which have same LJ parameters. In which case I cannot calculate now by hand how many are left

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants