NixOS-QChem
NixOS-QChem copied to clipboard
XTB 6.4.1 -> 6.5.1
Update to the most recent XTB version with many new features and bugfixes. XTB now requires 2 new dependencies:
- mctc-lib to unify input file parsing between different Grimme group programs. mctc-lib requires
- json-fortran to parse QCSchema input files
- test-drive to automate fortran tests in XTB
This seems to break pysisyphus:
FAILED tests/test_afir/test_afir.py::test_three_frag_afir - assert 2.62350526...
FAILED tests/test_modekill/test_modekill.py::test_modekill_xtb - AssertionErr...
FAILED tests/test_xtb/test_xtb.py::test_xtb_forces - assert 9.236406978009484...
= 3 failed, 678 passed, 190 skipped, 325 deselected, 10 xfailed, 62 warnings in 2622.60s (0:43:42) =
==================== Thresholds ====================
CN : 150.00000
rep : 500.00000
disp: 2500.00000
HB1 : 250.00000
HB2 : 450.00000
Pauling EN used:
Z : 1 EN : 2.20
Z : 8 EN : 3.44
electric field strengths (au): 0.000
-------------------------------------------------
| Force Field Initialization |
-------------------------------------------------
distances ...
----------------------------------------
generating topology and atomic info file ...
pair mat ...
computing topology distances matrix with Floyd-Warshall algo ...
making topology EEQ charges ...
#fragments for EEQ constrain: 1
----------------------------------------
generating topology and atomic info file ...
pair mat ...
computing topology distances matrix with Floyd-Warshall algo ...
making topology EEQ charges ...
#fragments for EEQ constrain: 1
rings ...
# BATM 0
# H in HB 2
atom neighbors erfCN metchar sp-hybrid imet pi qest coordinates
1 O 2 1.92 0.00 3 0 0 -0.571 -0.000000 -0.209404 0.000000
2 H 1 0.97 0.00 0 0 0 0.285 1.481500 0.837616 0.000000
3 H 1 0.97 0.00 0 0 0 0.285 -1.481500 0.837616 0.000000
#atoms : 3
#bonds : 2
#angl : 1
#tors : 0
#nmol : 1
#optfrag : 1
==================== Thresholds ====================
CN : 150.00000
rep : 500.00000
disp: 2500.00000
HB1 : 250.00000
HB2 : 450.00000
Pauling EN used:
Z : 1 EN : 2.20
Z : 8 EN : 3.44
electric field strengths (au): 0.000
-------------------------------------------------
| Force Field Initialization |
-------------------------------------------------
distances ...
----------------------------------------
generating topology and atomic info file ...
pair mat ...
computing topology distances matrix with Floyd-Warshall algo ...
making topology EEQ charges ...
#fragments for EEQ constrain: 1
----------------------------------------
generating topology and atomic info file ...
pair mat ...
computing topology distances matrix with Floyd-Warshall algo ...
making topology EEQ charges ...
#fragments for EEQ constrain: 1
rings ...
# BATM 0
# H in HB 2
atom neighbors erfCN metchar sp-hybrid imet pi qest coordinates
1 O 2 1.92 0.00 3 0 0 -0.571 -0.000000 -0.209404 0.000000
2 H 1 0.97 0.00 0 0 0 0.285 1.481500 0.837616 0.000000
3 H 1 0.97 0.00 0 0 0 0.285 -1.481500 0.837616 0.000000
#atoms : 3
#bonds : 2
#angl : 1
#tors : 0
#nmol : 1
#optfrag : 1
@eljost this is most likely also a problem for upstream pysisyphus. Could you check if pysis needs to be tweaked for XTB 6.5. They've changed topology handling in GFN-FF.
This commit https://github.com/eljost/pysisyphus/commit/e382c9dec9cc5bf4755414ea647a649322168bdb fixes pysisyphus it. I will include it as soon as it reaches master.
Is this fixed now in pysisyphus?
Yes, as part of this large merge commit to master: https://github.com/eljost/pysisyphus/commit/1b73e5dd167ce63a6ea9337544b73ad753467558 May be better to use a pin of master there for the moment than to build Frankenstein's patch.
pysisyphus still fails with the same error.
I still see the same error.
I still see the same error.
Yes, yes. But now the upstream CMake update also broke mctc-lib and test-drive. I have first fixed those. I am currently sitting down with @eljost and we are working through the failing tests. They are mostly numerical noise. I will ping you when done.
@markuskowa Here we are, all tests passing
@markuskowa Here we are, all tests passing
To good to be true ... Worked on pysis master but not in here ...
Now this has finally worked. Should be ready now.