crest-lab / crest

CREST - A program for the automated exploration of low-energy molecular chemical space.
https://crest-lab.github.io/crest-docs/
GNU Lesser General Public License v3.0
199 stars 42 forks source link

Not parsing flags from toml file #353

Open adamhorvath99 opened 2 weeks ago

adamhorvath99 commented 2 weeks ago

Dear all,

I might be wrong, but I have just updated to the 3.0.2 version of crest, and somehow the parsing of flags arguments from toml file is misteriously not working now. I use the --cinp flag but it is uneffective.


pprcht commented 2 weeks ago

You are using a weird setup that is mixing up options, and I don't think there is a bug:

If you tell me what exactly you want to do in all this, I might be able to better assist.

adamhorvath99 commented 2 weeks ago

Thank you very much! I am really lost at this point now.

I want two thing. First, do usual conf-analysis with crest using gfn2. Second, I want constrained conf-analysis with crest using gfn2 for TSs. I would need TOML files fro these. I think I misunderstood a lot of things in the manual

pprcht commented 2 weeks ago

If you do not explicitly need the xtb implementation and are fine with tblite you can either use the command line as

crest geom.xyz --gfn2 --cinp constraints.inp -T 28

or a toml file

input="geom.xyz"
threads=28
constraints="constraints.inp"
[[calculation.level]]
method="gfn2"
chrg=0
uhf=0

or a combination of both

crest input.toml --cinp constraints.inp -T 28

the point being, all new versions can work without an xtb binary.

adamhorvath99 commented 2 weeks ago

OK, now started to work. But I get this memory issue. I allocate 2GB per core and have a lot of scratch, I do not understand the problem.

In file '/home/runner/work/crest/crest/src/parsing/parse_xtbinput.f90', around line 720: Error allocating 2188448768777856 bytes: Cannot allocate memory

pprcht commented 2 weeks ago

This should have been fixed as of commit 5ca82feb2ec4df30a0129db957163c934f085952, are you using the current version? It might not be included in the conda build (yet)