Skip to content

Add TsLess sources #6

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add TsLess sources #6

wants to merge 1 commit into from

Conversation

robertodr
Copy link
Member

Add the tessellationless integration grid for the PCM as described in Pomelli 2004

Description

The GePol algorithm is based on a partition of the spheres that involves the use of spherical polygons. The TsLess algorithm uses a different strategy for the partition of the spheres that makes away with the polygons altogether in favor of just points and weights.

How Has This Been Tested?

  • Comparison of the number of points, cavity surface and cavity volume with reference values. This is done for a single sphere, the HF and the NH3 molecules. TESTS CURRENTLY BROKEN
  • Comparison of the total surface charge calculated with CPCM and IEFPCM with the exact value from Gauss' theorem. . This is done for a single sphere and the NH3 molecules. TESTS CURRENTLY BROKEN
    • We can compare our Leopardi partitioning of the spheres with the original EQP code
  • Comparison of QM energies with those published in the original article.

Todos

  • Developer Interest
  • Symmetry-handling in the TsLess Fortran backend.
  • Code duplication between the PEDRA and TsLess Fortran backends.
  • User-Facing for Release Notes
  • New cavity generator.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Questions

  • I think we should ditch GePol altogether in favor of TsLess for v2.

Status

  • Ready to go

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

Successfully merging this pull request may close these issues.

1 participant