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

Update atomic_density.ipynb #19

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

Conversation

FarnazH
Copy link
Member

@FarnazH FarnazH commented Nov 15, 2021

This PR is meant to finalize the atomic_density.ipynb notebook.

@Ali-Tehrani can you please include instructions on how to obtain the Slater density files?

@FarnazH
Copy link
Member Author

FarnazH commented Nov 15, 2021

P.S.: @Ali-Tehrani I noticed that the data exists in https://github.com/QuantumElephant/bfit/tree/master/bfit/data/data.tar. We should add comments on untaring data.tar.

@Ali-Tehrani
Copy link
Collaborator

P.S.: @Ali-Tehrani I noticed that the data exists in https://github.com/QuantumElephant/bfit/tree/master/bfit/data/data.tar. We should add comments on untaring data.tar.

I can't get your remote to push to this and github doesn't let me edit, we can add under step two " Compute Slater Density" the following:
"The data.tar folder in ./bfit/data/ folder needs to be untar, inside ./bfit/data/ folder in order to compute the atomic density for various atoms. See the [installation](https://github.com/theochem/BFit/) instructions in the README for more details."

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.

2 participants