atomate icon indicating copy to clipboard operation
atomate copied to clipboard

Add hashes to Q-Chem task documents

Open espottesmith opened this issue 2 years ago • 2 comments

The (relatively) new Materials Project molecules database (here) uses Weisfeiler-Lehman graph hashes, primarily as an aspect of molecule IDs. Currently, to add tasks to our build pipeline, I have a script to incorporate the hashes into completed Q-Chem task documents. This PR instead directly incorporates the hashes in task document construction at the time of drone assimilation.

Hashes have been incorporated in the Q-Chem drone tests, and these tests pass locally.

espottesmith avatar Dec 05 '23 16:12 espottesmith

So several tasks not related to graph hashes and Q-Chem docs failed. @samblau I think one of them is related to the fragmenter. Any ideas?

espottesmith avatar Dec 05 '23 17:12 espottesmith

Honestly, we should probably just deprecate / remove the fragmenter. In practice, it basically always made sense to do fragmentation in a script to customize it to a given application's species generation. I don't think the fragmentation workflow in Atomate really has any value, so if it's breaking tests, let's just get rid of it.

More broadly, I've stopped using Atomate in favor of QuAcc (and others have probably stopped using it in favor of Atomate2). I think Rishabh was working on Q-Chem stuff in Atomate2, which I think also included a new task doc schema? So it might be worth working in these changes there too.

samblau avatar Dec 06 '23 20:12 samblau