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

Feat/plot cleanup #138

Merged
merged 38 commits into from
Sep 15, 2023
Merged

Feat/plot cleanup #138

merged 38 commits into from
Sep 15, 2023

Conversation

cweniger
Copy link
Member

Closes #137 .

@cweniger cweniger requested a review from NoemiAM August 4, 2023 22:07
- plot_pair is again plot_2d (plot_pair should refer to some kind of corner plot, see arviz)
@cweniger cweniger removed the request for review from NoemiAM August 5, 2023 06:37
@cweniger cweniger assigned cweniger and unassigned NoemiAM Aug 5, 2023
@cweniger cweniger marked this pull request as draft September 1, 2023 18:38
@cweniger cweniger changed the base branch from master to rc0.4.5 September 1, 2023 18:38
 Conflicts:
	notebooks/0A - SwyftModule.ipynb
@cweniger cweniger marked this pull request as ready for review September 4, 2023 10:58
@cweniger cweniger marked this pull request as draft September 4, 2023 13:48
@cweniger
Copy link
Member Author

cweniger commented Sep 4, 2023

Before merging, I want to (re-)add the functionality to histogram the prior samples and smooth those before applying the likelihoods. That strategy should give proper posterior plots with far less samples, in particular for 2-dim posteriors, but will only work for 1-dim and 2-dim marginals. The codes are already there, but currently not used.

@cweniger cweniger marked this pull request as ready for review September 15, 2023 13:33
@cweniger cweniger merged commit 57b00eb into rc0.4.5 Sep 15, 2023
cweniger added a commit that referenced this pull request Sep 15, 2023
* Feat/better defaults (#142)

Updates:
* Modularisation of SwyftModule
* Restructuring of `configure_optimizers` and `configure_callbacks`
* New default is AdamW with early stopping
* Alternatives with AdamW and OneCycleLR and ReduceLROnPlateau exist as well
* Automatic reloading of best model at the end of training is default

* Feat/plot cleanup (#138)

Updates:
* Remove plot2.
* Extended options
* Add `smooth_prior` flag for interpolating likelihoods
* Renaming of plotting routines (plot_posterior, plot_corner, plot_pair)
* Update example notebook

---------

Co-authored-by: NoemiAM <noemi.anaumontel@gmail.com>
@cweniger cweniger deleted the feat/plot-cleanup branch September 15, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consolidate plotting routines
2 participants