ramess101 / JCED_FOMMS_Manuscript

0 stars 1 forks source link

Reviewer 1 Comment 5 #23

Open ramess101 opened 5 years ago

ramess101 commented 5 years ago

@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 commented 5 years ago

@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 commented 5 years ago

@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 commented 5 years ago

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 commented 5 years ago

@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 commented 5 years ago

@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 commented 5 years ago

@jpotoff So I am fine removing the discussion about agreement between fixed and flexible. But the NERD model actually was developed with flexible bonds, so we can’t exactly claim that we are using fixed bonds for the purpose of complying with the model specifications. But we can reiterate that we are verifying the method and our comparison data used fixed bonds.

On Saturday, March 2, 2019, Jeffrey Potoff notifications@github.com wrote:

@ramess101 https://github.com/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.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ramess101/JCED_FOMMS_Manuscript/issues/23#issuecomment-468983913, or mute the thread https://github.com/notifications/unsubscribe-auth/AWUvhJRS_xxMn5Z41fGHHzJrDRo6OlVOks5vS0BMgaJpZM4bSaiu .

jpotoff commented 5 years ago

@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 commented 5 years ago

@jpotoff Thats a good idea. I already have those data as well. I will make some plots comparing the results and include it in our response and maybe the SI.

On Saturday, March 2, 2019, Jeffrey Potoff notifications@github.com wrote:

@ramess101 https://github.com/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.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ramess101/JCED_FOMMS_Manuscript/issues/23#issuecomment-468987999, or mute the thread https://github.com/notifications/unsubscribe-auth/AWUvhLCOtgSwfnBPi-ViyIx4OCfBHHZkks5vS1G-gaJpZM4bSaiu .

ramess101 commented 5 years ago

@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 commented 5 years ago

@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 commented 5 years ago

@mrshirts @jpotoff @msoroush

Here is my response:

image image

msoroush commented 5 years ago

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

ramess101 commented 5 years ago

@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 commented 5 years ago

@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.