Skip to content

SEP_Fit-Spectra: Handle spectrum units and improve intensity_zero_guess handling - #166

Merged
jgieseler merged 16 commits into
soler-he:mainfrom
Christian-Palmroos:main
Aug 28, 2026
Merged

jgieseler merged 16 commits into
soler-he:mainfrom
Christian-Palmroos:main

Conversation

@Christian-Palmroos

Copy link
Copy Markdown
Contributor

No description provided.

@Christian-Palmroos

Christian-Palmroos commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Related to #161

Comment thread fit_spectra/make_the_fit.py Outdated


def MAKE_THE_FIT(spec_e, spec_flux, e_err, flux_err, ax, direction='sun', which_fit='best', e_min=None, e_max=None, g1_guess=-2., g2_guess=None, g3_guess=None, alpha_guess=5., beta_guess = 5, break_low_guess=0.065, break_high_guess=0.12, cut_guess = 0.12, I0_guess=None, exponent_guess = 2, use_random = False, iterations = 10, path = None, path2 = None, detailed_legend = False):
def MAKE_THE_FIT(spec_e, spec_flux, e_err, flux_err, ax, direction='sun', which_fit='best', e_min=None, e_max=None, g1_guess=-2., g2_guess=None, g3_guess=None, alpha_guess=5., beta_guess = 5, break_low_guess=0.065, break_high_guess=0.12, cut_guess = 0.12, I0_guess=None, E_0=0.1, exponent_guess = 2, use_random = False, iterations = 10, path = None, path2 = None, detailed_legend = False):

@jgieseler jgieseler Aug 20, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would assume that if I0_guess=None, then also E_0=None?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was accidentally undone along with all other progress, but should be there now!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean my question about E_0=0.1

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is exactly what I meant. It is set back to E_0 = None in the latest commit, which was accidentally undone in the previous commit.

@jgieseler jgieseler Aug 27, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My initial comment is older than the commit you're referring to. But I see now that you had changed this after my initial comment (without mentioning), and before the "previous commit"

@jgieseler

Copy link
Copy Markdown
Member

I assume the tests are failing because you have an older matplotlib version locally installed (they changes the font handling with 3.11); so you should update the baseline figures using a newer matplotlib version.

@jgieseler jgieseler changed the title Added "energy_unit", "intensity_unit", and"energy_zero/E_0" -parameters. SEP_Fit-Spectra: Handle spectrum units and improve intensity_zero_guess handling Aug 28, 2026
@jgieseler
jgieseler merged commit 2140b71 into soler-he:main Aug 28, 2026
69 of 76 checks passed
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