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

Extension and reimplementation of Shirley background substraction #36

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

MarcBHahn
Copy link

Hi
I reimplemented the section to substract a Shirley background as used in XPS.
It is now possible to substract a Shirley background only within the active region as it is often done in XPS. Additionally the implementation seems to be more stable when working with data having a bad SNR.
Best wishes
Marc

Marc Hahn and others added 3 commits May 20, 2020 19:37
Improves Shirley BG substraction by calculation of ymin and ymax parameter from the average of up to five surrounding datapoints.
This improves the Shirley BG estimation for XPS data with bad signal to noise ratio.
@wojdyr
Copy link
Owner

wojdyr commented May 30, 2020

Hi Marc,
that's a much needed feature. I searched my e-mail archive and I found a couple requests for selection of end points for Shirley background - the first one probably from 2013.
I have a couple of other pull requests waiting (for fityk and xylib). I'll get to it soon.

fityk/transform.cpp Outdated Show resolved Hide resolved
MarcBHahn and others added 8 commits October 18, 2021 19:54
Correct comments.
Adjust indices of Shirley background removal procedure for the analysis of XPS spectra.
Add the changes from main branch to prepare a new version fpr the pull request
Improve initial guess for shirleyBG determination to improve the starting values for the iterative procedure.
Add installation information about autogen.sh
merge updates from main
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