Skip to content
Snippets Groups Projects

Add some post processing tools

Merged Colm Talbot requested to merge add_some_post_processing_tools into master
All threads resolved!

This MR modifies how the marginal histograms are generated now:

  • everything in the posterior is plotted by default.
  • the pdf and cdf are plotted.

It also adds methods to the result to get the credible level at which injections are found and a function to make a pp-plot from many injections.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Colm Talbot resolved all discussions

    resolved all discussions

  • Colm Talbot added 1 commit

    added 1 commit

    Compare with previous version

  • Gregory Ashton
  • Hi @colm.talbot, could you expand on what is meant by

    everything in the posterior is plotted by default

    I couldn't quite see it from the code, do you mean only if plot_marginals is called? Or something else.

  • Author Maintainer

    Yeah, just if plot_marginals is called.

  • Colm Talbot added 20 commits

    added 20 commits

    Compare with previous version

  • Colm Talbot added 28 commits

    added 28 commits

    • 45df2a8e...349d7f1a - 21 commits from branch master
    • 8630ff95 - Merge branch 'master' into add_some_post_processing_tools
    • 32f779e1 - raise error if injection parameters are not specified
    • 935c1878 - add credible levels tests
    • fd6b1fa3 - update pandas referencing
    • c0993c09 - PriorSet->PriorDict
    • 433ef4ba - flake
    • 58999905 - Merge branch 'add_some_post_processing_tools' of git.ligo.org:lscsoft/bilby into…

    Compare with previous version

  • Colm Talbot added 1 commit

    added 1 commit

    Compare with previous version

  • Gregory Ashton resolved all discussions

    resolved all discussions

  • Gregory Ashton approved this merge request

    approved this merge request

  • Paul Lasky approved this merge request

    approved this merge request

  • merged

  • Paul Lasky mentioned in commit 5c59c6d9

    mentioned in commit 5c59c6d9

  • Please register or sign in to reply
    Loading