Skip to content

✨ parsers: Update parsing tools and tests for more EPW output files #24

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.

Sign up for GitHub

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

Jump to bottom
Open
ymzhang0 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

✨ parsers: Update parsing tools and tests for more EPW output files #24

ymzhang0 wants to merge 1 commit into from

Conversation

@ymzhang0
Copy link
Collaborator

@ymzhang0 ymzhang0 commented Jan 20, 2026

  • Added new parsing functions for dos, phdos, lambda, etc.
  • FIx regex in parse_stdout to support scientific notation with 'E' or 'D'.
  • Update test data files (aiida.dos, aiida.phdos, aiida.phdos_proj, etc.) .
  • Update and add regression tests for new and modified parsers (a2f, dos, phdos, lambda, etc.).

@ymzhang0
Update parsing tools and tests for EPW output files
9e004b7 
- Rename parse_epw_phdos_basic to parse_epw_phdos and update to return all phonon DOS columns.
- Improve regex robustness in parse_epw_max_eigenvalue to support scientific notation with 'E' or 'D'.
- Update test data files (aiida.dos, aiida.phdos, aiida.phdos_proj, etc.) to match current EPW output formats.
- Update and add comprehensive regression tests for new and modified parsers (a2f, dos, phdos, lambda, etc.).
@ymzhang0 ymzhang0 requested a review from mbercx January 20, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mbercx mbercx Awaiting requested review from mbercx

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ymzhang0