pace-neutrons / Pace-Project-Plan

0 stars 0 forks source link

Mantid: use Euphonic for force constant import to SimulatedDensityOfStates algorithm #166

Closed ajjackson closed 3 years ago

ajjackson commented 3 years ago

The SimulatedDensityOfStates Algorithm in Mantid reads frequencies and optionally eigenvectors from Castep output files, providing a few useful plotting options.

It would be helpful to permit import of force constants data using Euphonic's import features:

This will require packaging of Euphonic with Mantid, which will likely introduce some challenges of its own. By targeting this relatively simple and unused Algorithm we can solve those problems while avoiding entanglement with the more complex Abins Algorithm.

As such this soft-blocks Abins/Euphonic integration tasks #67 #68

ajjackson commented 3 years ago

Work has started on this, but it is possibly blocked by #167

ajjackson commented 3 years ago

This code is now reasonably functional, but cannot be accepted until the packaging situation is developed. https://github.com/mantidproject/mantid/pull/30760

This can now be considered blocked by #167

ajjackson commented 3 years ago

Mantid PR is in review, looks good. https://github.com/mantidproject/mantid/pull/30760