Skip to content

Fix phonon_bands for band structures with different paths in k-space #250

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

Merged
merged 3 commits into from
Nov 21, 2024

Conversation

janosh
Copy link
Owner

@janosh janosh commented Nov 21, 2024

  • breaking: branch_mode renamed to path_mode in phonon_bands()
  • Add new "strict" option to path_mode for handling k-path mismatches
  • Fix issue where k-path segments were incorrectly overlaid when band structures had different paths (unseen k-path segments are now appended to the right edge of the band structure plot)
  • Add more unit tests for path_mode functionality

@janosh janosh added fix Bug fix PRs phonons Phonon bands and DOS plots labels Nov 21, 2024
@janosh janosh merged commit 8bc182e into main Nov 21, 2024
49 checks passed
@janosh janosh deleted the fix-phonon-bands-different-kpaths branch November 21, 2024 21:32
janosh added a commit that referenced this pull request Mar 28, 2025
#250)

* fix different k-path segments being overlayed when band structures don't agree

no unseen segments are appended to the right end of the band structure

* phonon_bands rename keyword branch_mode to path_mode and add strict option

* add unit tests for phonon_bands path_mode keyword
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fix PRs phonons Phonon bands and DOS plots
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant