Conversation
- Fix ckpt_path and prediction_save_path in inference command - Replace arbitrary strings with block inputs instead
- Fix inference command and comment about escaping special characters - Replace all occurances of activation energy with barrier height for precision - Fix path to predictions.csv
|
This PR can be squashed into one commit with the commit message of the last commit |
timmyte
left a comment
There was a problem hiding this comment.
Dear @tonyzamyatin
Thank for updating the notebook. Looks good to me.
Please include and update (if necessary) the disclaimer from the previous version and then we are good to merge:
# Deep Learning of Reaction Barrier Heights with ChemTorch
----
**Disclaimer**
This community notebook was kindly supplied by [Anton Zamyatin](https://github.com/tonyzamyatin) and [Esther Heid](https://github.com/hesther) for OPI version 1.0.0.
----
|
done, I believe we can merge now |
|
Thanks for the last fix! Please make sure your PR is update with the target branch, as it still has conflicts. |
|
@tonyzamyatin I currently fail to execute the notebook, there we postpone the merge. The error: |
|
Hi @timmyte, did you successfully run the ChemTorch command specified in the notebook? It is not a cell but has to be manually exectued from the terminal (in the chemtorch directory) by the user. If there was an error or unexpected behavior please let me know! |
|
Hi @tonyzamyatin I tried to install ChemTorch as described in the notebook but in the |
Updated ChemTorch CLI commands in the "Deep Learning of Reaction Barrier Heights with ChemTorch" due to latest changes in the ChemTorch API.