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

Add Schwenke's gas phase H2O analytical potential #166

Merged
merged 16 commits into from
Jan 26, 2024

Conversation

danielhollas
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (a70a2cd) 92.37% compared to head (f339338) 92.28%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
- Coverage   92.37%   92.28%   -0.09%     
==========================================
  Files          43       45       +2     
  Lines        6084     6182      +98     
  Branches      741      746       +5     
==========================================
+ Hits         5620     5705      +85     
- Misses        452      465      +13     
  Partials       12       12              
Flag Coverage Δ
unittests 24.23% <0.00%> (-0.44%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/force_abin.F90 93.95% <ø> (ø)
src/forces.F90 97.54% <100.00%> (+0.02%) ⬆️
src/init.F90 90.53% <100.00%> (+0.02%) ⬆️
src/utils.F90 98.18% <ø> (ø)
src/h2o_schwenke.f 91.52% <91.52%> (ø)
src/force_h2o.F90 77.77% <77.77%> (ø)

@danielhollas danielhollas force-pushed the schwenke-water-potential branch from c808f6d to 176651a Compare November 10, 2023 18:20
@danielhollas danielhollas force-pushed the schwenke-water-potential branch from 176651a to cee8338 Compare January 19, 2024 14:00
@danielhollas danielhollas changed the title WIP: Add Schwenke s gas phase H2O potential Add Schwenke's gas phase H2O analytical potential Jan 26, 2024
Copy link
Contributor Author

@danielhollas danielhollas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mbarnfield63 It would be great if you could go through the changes in this PR, and make a comment where things are not clear to you. Focus on file force_h2o.F90. The actual potential is in h2o_schwenke.f, but that has been taken essentially verbatim from the paper (and we already went through it together once).
Changes to the other files are not that important, they are mostly just hooking up the new potential trough ABIN. (and there are a couple of unrelated changes as well).

I am going to merge this to the master branch, which you can then pull-in and start your own branch on your fork for the numerical forces.

Note that I have renamed the module and function names a bit to make them consistent, so your scan script would need to be updated accrodingly.


! TODO: Implement numerical forces generally for all potentials
! For now, they can be implemented here and hardcoded for a specific H2O potential
subroutine numerical_forces(x, y, z, Epot, fx, fy, fz, natom, nbeads)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mbarnfield63 Here's where you can start with your implementation.
Ultimately we'll want to have the implementation of numerical forces generalized and in a separate module, but that's not important at this stage.

@danielhollas danielhollas merged commit 911fbab into master Jan 26, 2024
19 checks passed
@danielhollas danielhollas deleted the schwenke-water-potential branch January 26, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant