adc-connect / adcc

adcc: Seamlessly connect your program to ADC
https://adc-connect.org
GNU General Public License v3.0
32 stars 19 forks source link

Implement PCM and other solvation models supported in backends #70

Open mfherbst opened 4 years ago

mfherbst commented 4 years ago

With #30 merged the next step in this direction is to go for PCM (and maybe EFP)?

maxscheurer commented 4 years ago

Having these features is not a problem at all. Testing and verifying them is another issue. Furthermore, not all backends support all of the solvation models. For PE, it was easy because I knew about the PE implementation in each program 😉

Some notes:

If we do not explicitly test the other solvation models, it's not a problem to support the feature and throw a warning that no reference data for these values is available or something.

mfherbst commented 4 years ago

Thanks for the info. I did not have in mind to do this right now. It was more of a mental note that this should not be too hard to get done with the structure we implemented.

Also, in principle the results might not be identical but should (hopefully) at least agree qualitatively for the different PCM implementations (might be naive ...). So we might not be able to compare to 6 digits, but I would expect for simple cases to at least find agreement to maybe the leading order(s). This allows us to find the usual factors of 2 or minuses, which would be a good start for debugging. For regression testing we can use hard-coded data again.