Rafael-G-C / pyDFTD3

Python version of Grimme's D3-dispersion correction for Gaussian input/output
http://www.patonlab.com
MIT License
3 stars 1 forks source link

Preparing for JAX #5

Closed robertodr closed 3 years ago

robertodr commented 3 years ago

List of TODOs to make the code differentiable with JAX

robertodr commented 3 years ago

Replying here to the Zulip chat:

  • [ ] Document input parameters to the d3 function. Related to #2.

The d3 function takes some input parameters to decide which flavor of the D3 model to use and which atom pairs to include/exclude. The inclusion/exclusion is mentioned in issue #2. It would be good if we document these parameters in a documentation string. This is not high priority though.

  • [ ] Rearrange input parameters to the d3 function as suitable for usage with JAX (see the jax_tests repository)

In jax_tests the function to differentiate only accepts atomic charges and coordinates. Here it takes those, plus other options. I believe that, in order to make d3 differentiable, we need to change the order in which parameters are passed.

robertodr commented 3 years ago

I think this is done, so we can move on to computing derivatives :confetti_ball: