Skip to content

Fix label in NEBAnalysis#4473

Merged
shyuep merged 12 commits intomaterialsproject:masterfrom
DanielYang59:fix-neb-plotter
Aug 13, 2025
Merged

Fix label in NEBAnalysis#4473
shyuep merged 12 commits intomaterialsproject:masterfrom
DanielYang59:fix-neb-plotter

Conversation

@DanielYang59
Copy link
Copy Markdown
Contributor

@DanielYang59 DanielYang59 commented Aug 2, 2025

To close #4465


Before:
Image

Now:
image

@DanielYang59
Copy link
Copy Markdown
Contributor Author

DanielYang59 commented Aug 2, 2025

@Asif-Iqbal-Bhatti fixed

A few small issues with your script though:

  • You might not need to create a mpl figure plt.figure(figsize=(8, 6)), or styles (plt.title/xlabel/ylabel) yourself
  • The unit of energies is in meV not eV

Something like this should work (if you don't debug):

import matplotlib.pyplot as plt
from pymatgen.analysis.transition_state import NEBAnalysis

neb_analysis = NEBAnalysis.from_dir("neb_plot_data", zero_slope_saddle=True)

ax = neb_analysis.get_plot(normalize_rxn_coordinate=False)

plt.show()

@DanielYang59 DanielYang59 marked this pull request as ready for review August 2, 2025 18:28
@DanielYang59 DanielYang59 marked this pull request as draft August 2, 2025 18:44
@Asif-Iqbal-Bhatti
Copy link
Copy Markdown

okay thank you for fixing.

@DanielYang59 DanielYang59 marked this pull request as ready for review August 2, 2025 18:50
@shyuep shyuep merged commit 12011a1 into materialsproject:master Aug 13, 2025
42 of 44 checks passed
@shyuep
Copy link
Copy Markdown
Member

shyuep commented Aug 13, 2025

Thanks,.

@DanielYang59 DanielYang59 deleted the fix-neb-plotter branch August 13, 2025 18:52
@DanielYang59
Copy link
Copy Markdown
Contributor Author

No problem

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.

In pymatgen.analysis.transition_state there is label issue

3 participants