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

About the directory libdftd4.a.p. #239

Closed
hongyi-zhao opened this issue Feb 27, 2024 · 1 comment
Closed

About the directory libdftd4.a.p. #239

hongyi-zhao opened this issue Feb 27, 2024 · 1 comment

Comments

@hongyi-zhao
Copy link

I noticed the following guidance here:

DFT-D4 (optional)

To include the DFT-D4 van der Waals method, install the library from the source on GitHub and add the following in the VASP makefile.include

CPP_OPTIONS += -DDFTD4
DFTD4_ROOT ?= /path/to/your/dft4/installation
LLIBS += -L$(DFTD4_ROOT)/build -ldftd4
INCS += -I$(DFTD4_ROOT)/libdftd4.a.p

First, it seems that the INCS line used above should be written as follows:

INCS += -I$(DFTD4_ROOT)/build/libdftd4.a.p

But even so, I'm still not sure if the INCS variable set above is equivalent the following:

INCS += $(shell pkg-config --cflags dftd4)

See here for the related discussion.

Regards,
Zhao

@awvwgk
Copy link
Member

awvwgk commented Feb 27, 2024

Please follow the updated instructions from the Vasp wike or the current dftd4 documentation. The content of the build directory should not be used for Vasp, instead use dftd4 after running install.

@awvwgk awvwgk closed this as completed Feb 27, 2024
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

No branches or pull requests

2 participants