Skip to content

refactor: migrate Boost.Python syntax to nanobind - #89

Draft
SYMSCAE wants to merge 25 commits into
diffpy:migration-nanofrom
SYMSCAE:nanobind-syntax
Draft

refactor: migrate Boost.Python syntax to nanobind#89
SYMSCAE wants to merge 25 commits into
diffpy:migration-nanofrom
SYMSCAE:nanobind-syntax

Conversation

@SYMSCAE

@SYMSCAE SYMSCAE commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

This PR is the initial port that migrates the original Boost.Python bindings to nanobind as mentioned in diffpy/pyobjcryst#88, noticed future improvements will be mentioned in TODOs but will be addressed in future PRs.

@SYMSCAE SYMSCAE changed the title Nanobind syntax refactor: migrate Boost.Python syntax to nanobind Jul 14, 2026
@SYMSCAE

SYMSCAE commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

@sbillinge Please see if this is what we want.

@stevenhua0320

stevenhua0320 commented Jul 25, 2026

Copy link
Copy Markdown

@SYMSCAE This PR is still failing. I suspect it is because we haven't updated the dependencies for this package (libdiffpy and pyobjcryst). So, in order to make it build properly, we might need to update its dependency so that we could test whether it would work. @sbillinge We might also open migration branch on libdiffpy and pyobjcryst so that @SYMSCAE and @joeseaer would work on that.

@sbillinge

Copy link
Copy Markdown
Contributor

@SYMSCAE This PR is still failing. I suspect it is because we haven't updated the dependencies for this package (libdiffpy and pyobjcryst). So, in order to make it build properly, we might need to update its dependency so that we could test whether it would work. @sbillinge We might also open migration branch on libdiffpy and pyobjcryst so that @SYMSCAE and @joeseaer would work on that.

Migration work in those repos is on migration-nano branches. I am not sure about pyobjcryst but we can check. Please coordinate with earlier work fine by @SYMSCAE and others (I don't remember exactly)

@sbillinge

Copy link
Copy Markdown
Contributor

@SYMSCAE @stevenhua0320 we need to merge this to move the 3D PDF calculator forward. Please can we work on getting the dependency issue resolved so we can move forward? This is a bit urgent.

@stevenhua0320

Copy link
Copy Markdown

@SYMSCAE Let's first work on pyobjcryst as I believe there is a migration-nano for migration purpose. For this week you could work on that package and bug me if you are ready for the review.

@SYMSCAE

SYMSCAE commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

@SYMSCAE Let's first work on pyobjcryst as I believe there is a migration-nano for migration purpose. For this week you could work on that package and bug me if you are ready for the review.

Hi Steven, I believe we don’t need to touch pyobjcryst. The only change in upstream library is libdiffpy, which is requiring a new conda release. After that we should be good to pass rest of the tests.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants