importance sampling posterior

[manuelgloeckler]

What does this implement/fix? Explain your changes

Adds the possibility of importance sampling.

Does this close any currently open issues?

Fixes #919

Any other comments?

• Need to add tests somewhere.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.30%. Comparing base (07e3995) to head (172d010).

❗ There is a different number of reports uploaded between BASE (07e3995) and HEAD (172d010). Click for more details.

HEAD has 1 upload more than BASE

| Flag | BASE (07e3995) | HEAD (172d010) | |------|------|------| |unittests|1|2|

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1183      +/-   ##
==========================================
- Coverage   81.76%   73.30%   -8.47%     
==========================================
  Files          93       93              
  Lines        7448     7458      +10     
==========================================
- Hits         6090     5467     -623     
- Misses       1358     1991     +633     

Flag	Coverage Δ
unittests	73.30% <100.00%> (-8.47%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
sbi/inference/posteriors/importance_posterior.py	86.15% <100.00%> (+31.46%)	⬆️
sbi/inference/snle/snle_base.py	93.81% <100.00%> (+0.19%)	⬆️
sbi/inference/snpe/snpe_base.py	90.96% <100.00%> (+2.00%)	⬆️
sbi/inference/snre/snre_base.py	94.44% <100.00%> (+0.15%)	⬆️

... and 23 files with indirect coverage changes

[gmoss13]

Looks good, thanks @manuelgloeckler! Make sure to fix the pyright error, I've marked what I think is causing it.

[manuelgloeckler]

Great, thanks for the pyright fix. Your suggestions resolved it.