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

add Dirac GOS for EELS quantification #8

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

Conversation

zezhong-zhang
Copy link

As mentioned in hyperspy/hyperspy#3399 (reply in thread), here adds the examples for using Dirac GOS for EELS quantification. It includes:

  • short text description and links to the databases
  • compare the cross-sections for different edges between the Dirac and DFT GOS
  • line profile comparison
  • elemental maps
    This is done in a very simple manner of passing GOS parameter during model creation and re-use of the previous quantification procedure and code (in a single jupyter cell), as I think the main purpose of the notebooks are showing users how to use exspy for quantification instead of detailed explanation of the GOS, but please let me know if further details are needed.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

Binder 👈 Launch a binder notebook on branch zezhong-zhang/exspy-demos/main

@ericpre
Copy link
Member

ericpre commented Sep 7, 2024

Thank you @zezhong-zhang for the pull request. We need a release of eXSpy before merging this PR - without the release, the notebook are failing when using the Dirac GOS.

The changes look good, maybe a few tweaks but we can do this once eXSpy has been released with the Dirac GOS.

@zezhong-zhang
Copy link
Author

Thanks @ericpre, no worries, we can finalise the tweaks after the exspy release.

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.

2 participants