Skip to content

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
@cweniger cweniger requested a review from NoemiAM September 1, 2023 18:39
 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

3 participants