Goodman-lab / DP5

Python workflow for DP5 and DP4 analysis of organic molecules
Other
173 stars 99 forks source link

NWChem no jobs to run #21

Closed AmonB closed 4 years ago

AmonB commented 4 years ago

I would like to know how to fix this problem.
Running NWChem DFT NMR calculations locally... There were no jobs to run. I've install NWchem, TINKER. Did I miss something?

image

KristapsE commented 4 years ago

Could you please post the initial command you used to launch the workflow? It doesn't look like the workflow was given any structure files to analyze...

AmonB commented 4 years ago

image image

I wonder my NWchem didn't work or there is something I miss in the PyDP4.py

KristapsE commented 4 years ago

You need to provide 3D structure candidates for the NMR spectra analysis to work - you should add them in the add structure box.

AmonB commented 4 years ago

Thank you, I added two .sdf but now I have another problem.
image

image I have added the path in the Tinker.py image

KristapsE commented 4 years ago

Could you please download the most recent source version? I think this is the issue in the GUI where you need to launch it from the folder where your input files are, otherwise it won't find them. We have done some bugfixing in the last two weeks, which may have fixed this issue as well. If not, then please for the time being launch the GUI from the folder containing your input files. The command line version does not have this issue.

AmonB commented 4 years ago

Thank you for your excellent project. I'm looking forward to the better version. And I wonder there will have the demo .py files that highlight where should be change in the next version, because there will be many tricky errors if the settings are wrong.

KristapsE commented 4 years ago

Ideally there would be a configuration script that fills in a config file, which is then read by PyDP4.py and PyDP4_GUI.py scripts on startup. Certainly on the TODO list - just like many other things!

HowarthA commented 4 years ago

Hi, AmonB thanks for getting in touch. I've noticed you're currently not giving DP4-AI the NMR data in quite the way it is expecting. At the moment DP4-AI wants to be given a single folder containing one/two subfolders, one named "Proton" containing the Proton NMR data and one called "Carbon" containing the carbon NMR data. Have a look at one of the files in the repository if you are still unsure https://doi.org/10.17863/CAM.47721 This being said we are currently testing a new file input system which will make all of this much easier, hope this helps!

KristapsE commented 4 years ago

Hi @AmonB! I had a chance to do some coding today and implemented a separate config file - settings.cfg. This meant as an easier way to let DP4-AI know where to find the molecular mechanics (TINKER or MacroModel) and the DFT (Gaussian or NWChem) packages. A sample settings.cfg file is provided in the latest update - feel free to download the latest version, test it and let us know how it does!

KristapsE commented 4 years ago

So we have made several improvements over the last week, the latest version shouldn't have these issues. If they persist, please open a new issue, but I am going to close this one.