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

Do you use the simulation cell to generate the .dat? #1

Open
Leticia-maria opened this issue Apr 3, 2022 · 5 comments
Open

Do you use the simulation cell to generate the .dat? #1

Leticia-maria opened this issue Apr 3, 2022 · 5 comments
Labels
enhancement New feature or request

Comments

@Leticia-maria
Copy link

Hi! Great project! I have a question: do you use the simulation cell to generate the data file? I didn't what the utility of the simulation cell here.

@stefanbringuier
Copy link
Member

Thanks for your interest!

So I think your question is referring to the cell line in the input file:

#Hydrogen molecule
#Angstroms
cell 10.0 10.0 10.0
natoms 2
#Atoms Z ,x , y, z in Angstroms
positions 1 0.00 0.00 0.00 &
1 4.4 0.00 0.00
#Basis definition
basistype 6-31
basisfile ../../examples/basissets/H.6-31G.mod

So this is there because I initially intended to extend the code to enable various transformations and allow for periodic systems. I never got around to this, and so it adds no utility.

@Leticia-maria
Copy link
Author

Leticia-maria commented Apr 8, 2022

Thank you for your reply, @stefanbringuier . I was talking about the type: SystemOfAtoms

struct SystemOfAtoms
    natoms::Integer;
    atoms::Array{Atom,1};
    cell::Array{Float64,2};
    function SystemOfAtoms(natoms=1,
                    atoms=[Atom()],
                    cell=zeros(3,3))
            if natoms < 0
                error("Simulation cell cannot have negative atoms.")
            end
            if size(cell) != (3,3)
                error("Simulation cell is not a 3x3 array.")
            end
            if typeof(atoms) != Array{Atom,1}
                error("Atomic entries are not of the typeof() == Atom.")
            end
            new(natoms,atoms,cell)
    end
end #SystemOfAtoms

@Leticia-maria
Copy link
Author

Leticia-maria commented Apr 8, 2022

I didn't get why have you used it @stefanbringuier

@Leticia-maria
Copy link
Author

Leticia-maria commented Apr 10, 2022

Any updates, @stefanbringuier ?

@stefanbringuier
Copy link
Member

@Leticia-maria so the cell field in SystemOfAtoms is not used, but the idea was that if the code base get's extended to allow for various transformations of the simulation, the cell info will be required. In addition if support is added for planewave basis the cell with periodic boundary conditions provides numerical conveniences.

@stefanbringuier stefanbringuier added the enhancement New feature or request label Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants