-
Notifications
You must be signed in to change notification settings - Fork 24
CREST Adapter #807
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
base: main
Are you sure you want to change the base?
CREST Adapter #807
Changes from 9 commits
c7e9c0a
c727294
e265cdd
432e594
5d1612d
8891252
146a145
abd9201
87f3c51
0baa098
cab5b28
f8d102b
24ac9db
9e7cab6
96c70ce
1b2a9cb
1b20560
a18828f
42b0798
63207c4
5c7dc77
c46792c
9840d33
15503d9
de8e67b
6ae8b8c
00b5e2f
a9ac64f
c26d5b2
7986864
b995e05
5e67ea4
3be97bf
c5b659f
a985a3c
b3b54ec
1267ea5
52c558d
9d86a9a
86d8f12
456bca8
41f0454
b948f75
7a4c61a
02cddd7
21f1880
3f21891
591b492
7425633
254a29f
78fc445
7261198
938c49b
801498b
1e71353
94aa7d5
f3bc145
dd796ed
abeabd2
a4cf6fb
994d6c4
701f986
58b8fd5
86b8044
d7258d3
a7e9f63
5646c5b
2dbae7a
eb83cc8
b8c6644
9e1b3fa
ef121b6
885fa2f
1a59773
8c68b90
b40f76e
c6b87c4
87ce966
9182a1a
0149310
3e8a7e0
d7a2916
38a7e0f
4febdd8
2ac4488
3ef6716
2ef5e15
d2d75cb
47e2b1b
2615bb6
dc2d4c0
5514eed
a5735a6
2747bf8
8d68edf
3271067
a415613
a63a1fe
4ae2bf2
ca7ce14
c4fe12a
ee0be10
f31bb29
ee6f73e
e8124ba
452f4c9
5d2227d
79610b5
a807c42
70ddb5f
2b93b77
de5f433
d9b92e9
b07f425
6c91833
3825f3e
3819ea0
ada4955
383351c
873447d
54dc1b7
4645936
caa1a66
ccb2473
7f9feec
3f58df6
ed3b588
018fd7d
9503e26
affa87d
9553303
df45582
ad57056
cead051
916d994
2c76e57
f10ea72
8467b15
48ae9d9
27463a1
9aba13a
56a704e
dc3b667
3688d7d
4694bd8
9244dda
9a9d85d
7d5a147
68c6ad2
6a6e78c
4c641a6
4afc7ee
286a458
9fbad2a
8661c5c
8e5431d
70f2062
e57b401
309a903
8f5e604
b027f1f
8b69560
4a34680
ac1323f
6e955d0
7de85bf
691d2f7
586f3ca
523f68f
7568c61
032ff53
84b13d5
a65839c
d7c6372
900ae79
49dedfb
c28bc04
eea2313
16e3ef2
66dd09b
443663c
da54518
d2ad09f
bf18c6c
57635d8
9a4e624
42fef30
6c513c3
dcb58bc
92986a1
42044bc
c2972c7
1b95a97
14d206e
f2e1ac2
e05ab42
e87e13d
be39a2b
a19fed3
e289b9b
dbdac44
4e10004
09f5e80
2611bb3
5e56a0b
92b5b80
10b5dff
b8d31f4
c2a8685
3db7e8d
aad1883
4946d61
9631d63
2b01ee2
5a240c0
8249e57
383f756
8176d4e
fb692d3
8722f90
5c7903e
0137529
35d9bd3
f7665c9
bf1a82d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| cdef double pi, Na, kB, R, h, hbar, c, e, m_e, m_p, m_n, amu, a0, E_h, F, E_h_kJmol, bohr_to_angstrom | ||
| cdef double pi, Na, kB, R, h, hbar, c, e, m_e, m_p, m_n, amu, a0, E_h, F, E_h_kJmol, bohr_to_angstrom, angstrom_to_bohr |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -41,7 +41,7 @@ | |
| 'Cyclic_Ether_Formation': ['kinbot'], | ||
| 'Cyclopentadiene_scission': ['gcn', 'xtb_gsm'], | ||
| 'Diels_alder_addition': ['kinbot'], | ||
| 'H_Abstraction': ['heuristics', 'autotst'], | ||
| 'H_Abstraction': ['heuristics', 'autotst', 'crest'], | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd like to see the implementation a bit differently: |
||
| 'carbonyl_based_hydrolysis': ['heuristics'], | ||
| 'ether_hydrolysis': ['heuristics'], | ||
| 'nitrile_hydrolysis': ['heuristics'], | ||
|
|
@@ -77,7 +77,8 @@ | |
| adapters_that_do_not_require_a_level_arg = ['xtb', 'torchani'] | ||
|
|
||
| # Default is "queue", "pipe" will be called whenever needed. So just list 'incore'. | ||
| default_incore_adapters = ['autotst', 'gcn', 'heuristics', 'kinbot', 'psi4', 'xtb', 'xtb_gsm', 'torchani', 'openbabel'] | ||
| default_incore_adapters = ['autotst', 'crest', 'gcn', 'heuristics', 'kinbot', 'psi4', 'xtb', 'xtb_gsm', 'torchani', | ||
| 'openbabel'] | ||
|
|
||
|
|
||
| def _initialize_adapter(obj: 'JobAdapter', | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,7 @@ | ||
| import arc.job.adapters.ts.autotst_ts | ||
| import arc.job.adapters.ts.crest | ||
| import arc.job.adapters.ts.gcn_ts | ||
| import arc.job.adapters.ts.heuristics | ||
| import arc.job.adapters.ts.kinbot_ts | ||
| import arc.job.adapters.ts.seed_hub | ||
| import arc.job.adapters.ts.xtb_gsm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did we add install-crest into install-all as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comment above