Skip to content

Swyft integration#686

Open
DevarshiChoudhury wants to merge 2 commits intomainfrom
552-integrate-swyft-for-truncated-marginal-neural-ratio-estimation
Open

Swyft integration#686
DevarshiChoudhury wants to merge 2 commits intomainfrom
552-integrate-swyft-for-truncated-marginal-neural-ratio-estimation

Conversation

@DevarshiChoudhury
Copy link
Copy Markdown
Member

No description provided.

@DevarshiChoudhury DevarshiChoudhury changed the title Added a notebook for using swyft and updated gitignore Swyft integration Mar 6, 2026
@sguillot
Copy link
Copy Markdown
Contributor

sguillot commented Mar 6, 2026

@DevarshiChoudhury
Failing to import swift. It is installed (swyft v0.4.5), but the import is calling scipy.integrate.simps which does not exist. Only scipy.integrate.simpson exists.

Perhaps, this is a version problem. I have scipy v1.17.0
Which one do you have.

@drannawatts
Copy link
Copy Markdown
Member

The dependence on an older version of scipy is mentioned here : undark-lab/swyft#151

Christoph mentions a new framework in the making, we should probably discuss this with him.

@thjsal
Copy link
Copy Markdown
Contributor

thjsal commented Mar 16, 2026

Certainly at least some cleaning should be done to the new tutorial notebook. Now there is a huge overflow of printings with "Drawing samples from the joint prior..." (see https://github.com/xpsi-group/xpsi/blob/625184ca89f49b18242a995e71006ede3e003ce2/docs/source/xpsi_swyft.ipynb). And then it should also be added to the index and "Start here" pages.

@drannawatts
Copy link
Copy Markdown
Member

Removed from the upcoming version release for now pending discussions with Swyft developers and more general thinking about SBI implementations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate swyft with X-PSI for Truncated Marginal Neural Ratio Estimation (TMNRE)

4 participants