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

QCSchema Error handling #33

Open
jthorton opened this issue Oct 26, 2020 · 2 comments
Open

QCSchema Error handling #33

jthorton opened this issue Oct 26, 2020 · 2 comments
Labels
qca Related to the QCArchive infrastructure xtb Related to the upstream xtb dependency

Comments

@jthorton
Copy link
Contributor

jthorton commented Oct 26, 2020

Describe the bug

When running GFNFF in the public QCArchive via QCEngine we have hit some errors but we unable to view the error message as creating the QCElemental atomic result seems to raise a validation error (see the message below). I think any time xtb fails and the return_result is set to 0 it needs to just be the correct length to be reshaped by the validator here and should depend on the driver requested. A full list of optimization IDS with the issue can be found here.

To Reproduce

Our xtb worker conda environment can be found here.

The error message from a gredient calculation is:

File "/home/chodera/miniconda/envs/qcfractal/lib/python3.7/site-packages/xtb/qcschema/harness.py", line 208, in run_qcschema
return qcel.models.AtomicResult(**ret_data, stdout=output)
File "pydantic/main.py", line 346, in pydantic.main.BaseModel.init
pydantic.error_wrappers.ValidationError: 1 validation error for AtomicResult
return_result
cannot reshape array of size 1 into shape (3) (type=value_error)

The inputs are recorded in the public QCArchive the dataset with the issues is a Torsiondrivedataset with the name OpenFF-benchmark-ligand-fragments-v1.0.

Expected behaviour

The error message should be recorded in the result.

@jthorton jthorton added the unconfirmed This report has not yet been confirmed by the developers label Oct 26, 2020
@awvwgk
Copy link
Member

awvwgk commented Oct 26, 2020

Thanks for testing the QCEngine integration of xtb. I'm happy to see that GFN1-xTB and GFN2-xTB seem to work reliably via the API.

GFN-FF instead it is currently only supported experimentally via the xtb C-API, therefore this project provides an interface but it is not yet as stable or robust as the tight binding methods. This means I cannot guarantee that the C-API will actually be able to propagate errors back correctly to the caller side in case of GFN-FF calculations, which is quite unfortunate.

For the GFN-FF to work reliably in QCEngine there is definitely some work required in the C-API of xtb first.

@awvwgk awvwgk added qca Related to the QCArchive infrastructure xtb Related to the upstream xtb dependency and removed unconfirmed This report has not yet been confirmed by the developers labels Oct 26, 2020
@jthorton
Copy link
Contributor Author

Yes, they have worked fantastically so thanks for making this possible!

I see, please let me know if you would like help to pull down the optimizations and run them locally to try and reproduce the errors if this would help the development.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qca Related to the QCArchive infrastructure xtb Related to the upstream xtb dependency
Projects
None yet
Development

No branches or pull requests

2 participants