Skip to content

Fixed remaining issues for IAW fitting#85

Merged
almilder merged 48 commits into
mainfrom
load_from_file
Mar 26, 2025
Merged

Fixed remaining issues for IAW fitting#85
almilder merged 48 commits into
mainfrom
load_from_file

Conversation

@almilder
Copy link
Copy Markdown
Collaborator

Also updated a number of the tests with some that still need updates as they were created with the parameter initialization bug

joglekara and others added 30 commits January 24, 2025 14:36
…tions initialized using the Mora Yahi formulation
parameterized by LTx and LTy now
fixed EDF axes storage
surfaced all parameters of bg algorithm
fixed case sensitivity for bg type
move calculation of used points to postprocessing
switched plotting and parameter logging back to fitted parameters only
reenable sequential fitting
Updated loss calculations to postporocess with chi squared
Fixed plot scaling issue
Updated env files to prevent package issues
Updated calibrations for 106310
Added casefold to background naming on ions
@almilder almilder requested a review from joglekara March 22, 2025 21:24
Copy link
Copy Markdown
Member

@joglekara joglekara left a comment

Choose a reason for hiding this comment

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

Looks fine. Some things require removal and tidying up

Comment thread .gitignore Outdated
Comment thread arts2d_fwd.ipynb Outdated
Comment thread test_arts1d_random.py Outdated
Comment thread test_arts2d_random.py Outdated
Comment thread tsadar/inverse/loss_function.py Outdated
Comment thread tsadar/inverse/loss_function.py Outdated
Comment thread tsadar/utils/process/postprocess.py Outdated
@almilder almilder requested a review from joglekara March 26, 2025 14:01
Copy link
Copy Markdown
Member

@joglekara joglekara left a comment

Choose a reason for hiding this comment

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

Looks good, merge away!

@almilder almilder merged commit 81f667c into main Mar 26, 2025
@joglekara joglekara deleted the load_from_file branch March 26, 2025 16:47
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