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.

Sign up for GitHub

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

Jump to bottom

Kde loss dev #126

Open
wants to merge 43 commits into
base: main
Choose a base branch
from
Open

Kde loss dev #126

wants to merge 43 commits into from

Conversation

aphearin
Copy link
Contributor

WIP PR building a diffmahpop optimization based on kdescent

aphearin and others added 30 commits July 15, 2024 11:43
@aphearin
Add kdescent to testing env
ec88f83
@aphearin
Begin to build unit test of kdescent application
4cc5d72
@aphearin
flake8 fix
1fff82b
@aphearin
Add failing assert to check workflow setup
e7b2a72
@aphearin
Add test that should fail if kdescent is not installed
4d46271
@aphearin
Fix import issue
8575962
@aphearin
axopt --> optax
29ca8a3
@aphearin
Remove temporary assert
59fd622
@aphearin
Concatenate data
e3dd3a3
@aphearin
@AlanPearl I have a working call to compare_kde_counts but the value …
8c1b015 
…of the loss is unexpectdly gigantic
@aphearin
Loss is gigantic even when fiducial and proposed models are extremely…
ddc3025 
… similar
@aphearin
Resolve my confusion over magnitude of kde-based loss
4924292
@aphearin
Restore large excursion in u_param space
76b4eb7
@aphearin
Add single-constraint loss function
222a656
@aphearin
Add convenience function get_single_sample_self_fit_target_data
543c12b
@aphearin
Add multi-sample loss for self-fit data
d140396
@aphearin
add failing test of kdescent.adam
8b76b14
@aphearin
Increase robustness of test_single_sample_kde_loss_self_fit, which no…
8d05efb 
…w fails due to NaN-valued loss
@aphearin
test_single_sample_kde_loss_self_fit passes now with num_target_redsh…
8b9438c 
…ifts_per_t_obs = 3. Previously failed with num_target_redshifts_per_t_obs = 10
@aphearin
Start ignoring tests based on previous implementation that uses Mh(t)…
e35b043 
… as n-dim data
@aphearin
Start building 1-d kde optimizer
4ba4e4e
@aphearin
Add new implementation of single_sample_kde_loss_self_fit based on 5 …
9db6911 
…hard-coded 1-d kde pdf losses
@aphearin
New unit test of multi-obs loss passes
2098f4e
@aphearin
New unit test of kdescent.adam passes
a5c6ce3
@aphearin
Merge pull request #128 from ArgonneCPAC/kde_loss_dev_1d
2efee6a 
Kde loss dev 1d
@aphearin
Begin to develop new module kde2d_wrappers.py
d5eb239
@aphearin
Implement 2d single_sample_kde_loss_self_fit. Next steps are to fill …
ce7edad 
…out the remaining kernels, and then switch to logarithmic sMAR
@aphearin
Add remaining kernels
6ff8c29
@aphearin
Switch to logarithmic specific mass accretion rate
f8a0596
@aphearin
Fix bug in test_mc_diffmah_preds
0ade3b8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@aphearin