Prob_Angle Segmentation Fault or Bug?

More
7 years 2 months ago #296 by byoo1
Hi Josh,

Thanks for finding this out. Looks like the internal coordinates for an inserted molecule were not being updated in the insertion move routine. (This should only be an issue for the angle routine.)

I will go ahead and upload the hotfix (including the one you mentioned regarding the 'angle_to_move'). There should be a new tarball uploaded onto the website later today. I'll post a comment here once it's done.

Also, let us know if you run into any other issues with the angle or dihedral routines!

-Brian
The following user(s) said Thank You: jdmoore

Please Log in to join the conversation.

  • jdmoore
  • jdmoore's Avatar Topic Author
  • Offline
  • Junior Member
  • Junior Member
More
7 years 2 months ago #297 by jdmoore
Thanks Brian,

It seems to work so far.

Josh

Please Log in to join the conversation.

  • jdmoore
  • jdmoore's Avatar Topic Author
  • Offline
  • Junior Member
  • Junior Member
More
7 years 2 months ago #300 by jdmoore
Hi Brian,

So far in the updated version, I'm not finding any runtime errors with GCMC, but with NPT, I'm sometimes seeing

Attempted to delete part of molecule 375 of species 1
but the molecule energy is too high
Try increasing Rcutoff_Low, increasing the box size, or
decreasing the initial number of molecules
This error occurred in subroutine Cut_N_Grow on step 238.
Fatal Error. Stopping program.

This is when I turn Prob_Angle on. If I do not have Prob_Angle, I don't see this error.

I can do what it says, and decrease the number of initial molecules and then it will run.

Is this expected with Prob_Angle? If I keep everything except Prob_Angle, including Prob_Dihedral, it runs ok.

I'm attaching an example which generates the error within 100 steps.

Thanks.

Josh
Attachments:

Please Log in to join the conversation.

More
7 years 2 months ago #303 by byoo1
Hi Josh,

I'll take a look at this and get back to you asap. Thanks for attaching the files!

-Brian

Please Log in to join the conversation.

More
7 years 2 months ago #304 by byoo1
Hi Josh,

It looks like an overlap flag is getting triggered in the fragment_growth.f90 on line 1894.

1894 overlap_trial(ii) = .TRUE.

Looks like we removed that line in our version but not in the release version. For a quick fix, just delete that line and recompile the code. We will upload the hotfix soon.

-Brian

Please Log in to join the conversation.

More
7 years 2 months ago #305 by byoo1
The hotfix has been uploaded. I've run your systems (NPT for isooctane and GCMC for pentane) for 100000 sweeps and had no issues.

-Brian

Please Log in to join the conversation.

Time to create page: 0.122 seconds