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

Path to gap_files is not added in new make_gap_files.py #4

Open
Rinochek opened this issue Mar 25, 2022 · 8 comments
Open

Path to gap_files is not added in new make_gap_files.py #4

Rinochek opened this issue Mar 25, 2022 · 8 comments

Comments

@Rinochek
Copy link

Hi,

in the generated potential.gap file, the path to gap_files/ is not added. For example,
gap_beg distance_2b
species1 = H
species2 = H
delta = 0.5
sigma = 0.5
rcut = 4.0
desc_sparse = "CH.xml.sparseX.GAP_2022_3_16_120_6_53_52_4881"
alphas_sparse = "alphas_distance_2b_1.dat"
gap_end

@richardjana

@richardjana
Copy link
Collaborator

@Rinochek Try giving an output file that starts with "gap_files/", e.g. "gap_files/CH.gap". The script should make the directory and place all the files there (see lines 193-199).

@Rinochek
Copy link
Author

Rinochek commented Mar 25, 2022

@richardjana
This partially solves the problem, all sparse files remain without a path. The short example of the output:

gap_beg distance_2b
species1 = H
species2 = H
delta = 0.5
sigma = 0.5
rcut = 4.0
desc_sparse = "CH.xml.sparseX.GAP_2022_3_16_120_6_53_52_4881"
alphas_sparse = "gap_files/alphas_distance_2b_1.dat"
gap_end

@richardjana
Copy link
Collaborator

Well, yes. Those sparse files are your original files. The script could move/copy them into the directory ... but I am not sure if that should be the expected behavior. @mcaroba What is your opinion on where the sparseX files should be?

@mcaroba
Copy link
Owner

mcaroba commented Mar 25, 2022

The canonical behavior should be that gap files are always in a gap_files directory. This is not meant to be left as a user's choice.

@mcaroba
Copy link
Owner

mcaroba commented Mar 25, 2022

@richardjana I gave you write access to the repo. Don't bother making a pull request, just push the changes directly when you've fixed the script.

@richardjana
Copy link
Collaborator

So, CH.gap should be placed in the current directory, all alphas files in gap_files, right? Is the script supposed to copy the sparseX files into gap_files?

@mcaroba
Copy link
Owner

mcaroba commented Mar 25, 2022

Everything should be in the gap_files directory, including the sparse files and the original .xml file.

@richardjana
Copy link
Collaborator

richardjana commented Mar 25, 2022

Except the .gap file, right?

Edit: Never mind. It should also be placed there, so that all the paths are actually working. Before, they only worked when the .gap file was in the current working directory (directly above gap_files).

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

3 participants