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

Reviewer 1 Comment 5 #23

Open
ramess101 opened this issue Feb 26, 2019 · 14 comments
Open

Reviewer 1 Comment 5 #23

ramess101 opened this issue Feb 26, 2019 · 14 comments

Comments

@ramess101
Copy link
Owner

@mrshirts @jpotoff @msoroush

On page 6 in the force fields section, authors mention that the bond stretching potential is neglected for all the force fields. They compared their result with those from Mick et al. but Mick also used fixed bond models. This is not valid justification. Have the authors conducted any tests showing that the bond stretching has no influence on the simulation result or is this reported from literatures in the similar systems? The same question comes to electrostatic interactions. Please show evidence that it is valid to ignore these interactions.

We compare with Mick et al because we are validating that our MBAR results agree with HR. There would actually be much less justification if we compared MBAR results that used fixed bonds with HR results that used flexible bonds, or vice versa. But I think the reviewer's point is really aimed at whether VLE calculations, in general, are sensitive to flexible or fixed bonds. I know there have been a few (older) studies that looked into the impact of bond treatment. I will try to find those. @jpotoff @msoroush can you think of any studies off the top of your head?

The question regarding electrostatic interactions is somewhat misplaced. One could argue that performing simulations with fixed or flexible bonds is a set-up choice (not strictly tied to the force field) whereas electrostatics are completely a model choice when fitting a force field. All UA or AUA models, that I am aware of, do not use point charges for normal, branched, and cyclic hydrocarbons. I don't think we need to vindicate this decision that has been made for the
OPLS-UA, TraPPE, NERD, AUA4, TAMie, and MiPPE force fields. Any thoughts?

@jpotoff
Copy link
Collaborator

jpotoff commented Feb 26, 2019

@ramess101 Well, both comments from the reviewer miss the mark. We can't do a validation of a method by changing the models used in the calculations.

For the alkynes, we did extra calculations to show that adding electrostatic interactions did not substantially improve the model for pure or mixture phase behavior (it's in the supporting info). If it's not going to effect alkynes, is definitely not going to be an issue for alkanes (for VLE). Could electrostatics and polarizability be important for the free energy of solvation of alkanes in water? Maybe, but that's not the model, or the properties of interest here.

@ramess101
Copy link
Owner Author

@jpotoff

We can't do a validation of a method by changing the models used in the calculations.

Very well said. I will try to emphasize this point in my response. I will also mention your experience with alkynes.

@mrshirts
Copy link
Contributor

Right, the point is not to determine whether bond stretching affects the results. The goal is to show that the approach allows you to determine GCMC-determined properties at a large number of parameters very quickly.

@ramess101
Copy link
Owner Author

@mrshirts @jpotoff @msoroush

Sorry for the delay. Here is my response for this reviewer comment. Please let me know if you have any objections, suggestions, or comments.

In particular, @jpotoff and @msoroush do you object to me mentioning that flexible bonds were not yet available in GOMC? Although the real reason we used fixed bonds for NERD was to compare our results with Mick et al., perhaps the underlying reason Mick et al. utilized fixed bonds was because flexible bonds were not supported by GOMC (?). Or do you know of a study that definitively demonstrates that flexible bonds do not impact phase equilibria?

image

@jpotoff
Copy link
Collaborator

jpotoff commented Mar 3, 2019

@ramess101 it seems to me that it would be more straightforward just to say the models used fixed bonds and remove any discussion of flexible bonds and their potential effect on the predicted phase behavior. This is a paper on a method, so we use the models as originally developed to validate the method.

@ramess101
Copy link
Owner Author

ramess101 commented Mar 3, 2019 via email

@jpotoff
Copy link
Collaborator

jpotoff commented Mar 3, 2019

@ramess101 We have tons of data for NERD from the branched alkanes paper, so if we need to address the fixed vs. flexible thing, we can provide a comparison between our results and the original NERD papers to show they are the same.

@ramess101
Copy link
Owner Author

ramess101 commented Mar 3, 2019 via email

@ramess101
Copy link
Owner Author

@jpotoff

Haven't gotten around to the comparison plots yet. But here is how I modified the paragraph. I basically just removed the sentence about agreement between fixed and flexible. I am considering add a line at the end saying, "We verify that fixed and flexible bonds yield indistinguishable results in supporting information."

image
image

@ramess101
Copy link
Owner Author

ramess101 commented Mar 5, 2019

@jpotoff @msoroush

I have plotted the Mick et al. and Nath et al. values for the NERD potential for branched alkanes. Nath et al. only reported orthobaric densities, so that is all we can compare. The agreement is within the uncertainties for liquid density at all state points (although 2-methylbutane and 2,4-dimethylhexane appear to have a bias in rhol). The rhov agreement is not as good, but fairly reasonable. This is not totally unexpected as my experience is that GCMC-HR and GEMC typically show some non-negligible deviations in the vapor phase properties even if both are run with fixed bonds.

When implementing the NERD potential previously, I recall that Mohammad pointed out that the parameter types are somewhat ambiguous. So maybe Mick et al. didn't use the exact same parameters for some of these compounds. Either way, I think this should be good enough to show that the phase equilibria are pretty similar.

image

@ramess101
Copy link
Owner Author

@mrshirts @jpotoff @msoroush

Here is my response:

image
image

@msoroush
Copy link
Collaborator

msoroush commented Mar 5, 2019

@ramess101 You might wanna avoid reporting data for the compounds that we were not sure which CH3 parameter to use.

@ramess101
Copy link
Owner Author

@msoroush

Yeah, that's true. I was just trying to maximize the amount of data I could plot here.

This is all that we are sure about in the NERD manuscript:

image

@ramess101
Copy link
Owner Author

ramess101 commented Mar 5, 2019

@msoroush

So we know isobutane (2-methylpropane) and 2-methylbutane are correct. 2-methylbutane was one of the bigger discrepancies. 2,4-dimethylhexane is the other concern, and it could be slightly more ambiguous to determine the parameters.

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

4 participants