Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initiate rebased branch for lattice dynamics #789

Merged
merged 207 commits into from
Mar 12, 2024

Conversation

Zhuoying
Copy link
Contributor

@Zhuoying Zhuoying commented Mar 5, 2024

Summary

After solving the branch conflicts between lattice-dynamics and main, here is a clean working tree.

The workflow of lattice dynamics (using HiPhive for Force Constants fitting) and all post-analysis to get thermal properties (thermal expansion coefficient, lattice thermal conductivity, vibrational free energy, etc)
The frame of this workflow is pretty well-established and can be used for production runs compatible with an older version of MP package toolkits (pymatgen, spglib, fireworks, custodian)

TODO

<If this is a work-in-progress, write something about what else needs
to be done>

  • Update to the latest MP environment (compatible with pmg etc)
  • Enable automatic post-analysis if ShengBTE fails to compute the thermal conductivity
  • Enable automatic NAC+LRC correction for ionic crystals
  • Provide notebook tutorial for WF submission

rees-c added 30 commits March 4, 2024 18:11
Added functionality for constraining supercell to a min number of atoms on top of the X times NN distances.
…l was not writing files in format that was readable by ShengBTE.
…required param (for putting poscars inside displacement folders)
@Zhuoying Zhuoying merged commit 2e541f2 into hackingmaterials:main Mar 12, 2024
1 check failed
@Zhuoying Zhuoying deleted the fix-ld-wf branch March 12, 2024 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants