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

cabintry integration #10

Open
lukasheinrich opened this issue Mar 24, 2021 · 1 comment
Open

cabintry integration #10

lukasheinrich opened this issue Mar 24, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@lukasheinrich
Copy link

lukasheinrich commented Mar 24, 2021

hi,

I just noticed that simplify doesn't actually use cabinetry (as I assumed purely looking at the plots) but reuses some of that code with slight modifications and I was curious whether things were missing from the cabinetry API / which modificatoins were necessary.

If @alexander-held agrees maybe it makes sense to try to upstream some of these modifications and then add cabinetry as a dependency

(I could also see an argument of simplify being a pure likelihood translator but the diagnostics seem nice to have)

@eschanet
Copy link
Owner

Hi @lukasheinrich,

wow, only took me 2 months to notice this comment ...

You're exactly right, I took some of the cabinetry plotting code for the diagnostics stuff. I'll check how easy/sensible it would be to upstream this to cabinetry.

The argument of understanding simplify to be a simple translator can of course be made, but I always found some minimal diagnostics quite helpful. I didn't plan to have anything fancier than simple pull plots, correlation matrices and yields tables though (so really just to make sure the full likelihood that goes in behaves the way you'd expect it to).

@eschanet eschanet self-assigned this May 26, 2021
@eschanet eschanet added the enhancement New feature or request label May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants