aangelopoulos / ppi_py

A package for statistically rigorous scientific discovery using machine learning. Implements prediction-powered inference.
MIT License
194 stars 14 forks source link

Add alpha parameter to ppi_mean_ci call #5

Closed PierreBoyeau closed 7 months ago

PierreBoyeau commented 7 months ago

The alpha parameter did not propagate when lhat=None. This PR is a quickfix to make sure it does.

aangelopoulos commented 7 months ago

You're a hero, Pierre. Thank you <3 Can you do the same for the other CIs, if it is missing? Then I'll merge.

PierreBoyeau commented 7 months ago

I had a quick look at the rest of the codebase, I did not find any similar issues elsewhere.

aangelopoulos commented 7 months ago

Great. I agree. MERGE!