-
Notifications
You must be signed in to change notification settings - Fork 652
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
[nnx] fix nanobind #4413
base: main
Are you sure you want to change the base?
[nnx] fix nanobind #4413
Conversation
1da88f8
to
6c135f7
Compare
cafa8c7
to
f19b1a8
Compare
6c135f7
to
9578456
Compare
f19b1a8
to
cf8fbfc
Compare
tests/flaxlib_test.py
Outdated
@@ -15,11 +15,10 @@ | |||
|
|||
# TODO: Re-enable this test after setting up CI build for flaxlib CC. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this line please, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the entire file
6258bc9
to
32bbddf
Compare
32bbddf
to
9f2ef36
Compare
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
6ed8dac
to
d088e18
Compare
d088e18
to
c7d0756
Compare
e7b8a0d
to
8241d18
Compare
8241d18
to
9ef4814
Compare
What does this PR do?
scikit_build_core.build
for packaging as detailed in: https://nanobind.readthedocs.io/en/latest/packaging.html_graph_flatten
and some helper types.