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

[WIP] Behaviour with Korg.Fit.ews_to_stellar_parameters when no EW errors given #323

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

andycasey
Copy link
Collaborator

The documentation regarding measured_EWs for Korg.Fit.ews_to_stellar_parameters says that if no EW errors are specified, all lines are assumed to have the same uncertainty. This is kinda true but leads to some unusual behaviour.

All lines are assumed to have the same uncertainty in EW of 1 mA (measured_EW_err=ones(length(measured_EWs))).

This sets equivalent uncertainties in EW, which is not the same as equivalent uncertainties in abundance, because strong lines (e.g., 200 +/- mA) end up getting weighted more than weak lines (e.g., 20 +/- 1 mA).

This is a WIP for now because I think Adam would prefer a default_EW_error function to set the default value.

@ajwheeler
Copy link
Owner

Do you still believe in this given what we now know about EW fitting?

@andycasey
Copy link
Collaborator Author

I do because the estimated error on EW is usually fractional (a percentage), but I'd like to do some science tests with the revised EW methods before strictly proposing how EWs are handled when no uncertainties are given.

If you are keen to do housekeeping on GitHub issues then you can close this PR and add it to an item in the roadmap/TODO list.

@ajwheeler
Copy link
Owner

Happy to leave it open as long as we still want to investigate.

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