-
Notifications
You must be signed in to change notification settings - Fork 132
Open
Description
To do with lower priority:
- Increase the transparency for reasons why the
ElectrodeInsertionMakerfails. Right now, the flow can stop in multiple places and the reasons aren't clear to the end user - Failure type 1: If no topotactic insertions were found (structural framework after insertion changed) then the
get_min_energy_summaryreturnsNoneandget_stable_inserted_resultsfails. Should throw an error inget_stable_inserted_results - Failure type 2:
InsertionElectrodeDoc.from_entriesreturnsNoneif it cannot process the flow output. That causesget_insertion_electrode_docto fail with a schema error. Should return a minimalInsertionElectrodeDocwith failure reasons
There are probably others - would appreciate if anyone using these flows could comment!
Metadata
Metadata
Assignees
Labels
No labels