Skip to content

Switching Github CI workflow to use nanobind. This should fix the err… - #3

Merged
Ra4ster merged 8 commits into
mainfrom
bindings/nanobindCI
Sep 3, 2026
Merged

Switching Github CI workflow to use nanobind. This should fix the err…#3
Ra4ster merged 8 commits into
mainfrom
bindings/nanobindCI

Conversation

@Ra4ster

@Ra4ster Ra4ster commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Description

Moved CI workflow to nanobind from pybind.

Type of change

  • Bug fix
  • New feature
  • Performance improvement
  • Documentation
  • Build/tooling
  • Breaking change (existing C++ or Python API behavior changes)

Testing

Working to correct these:

  • python build.py Release builds and DeepityTests passes
  • python build.py Debug builds and DeepityTests passes (required for anything correctness-sensitive)
  • pyright passes (if you touched bindings/, pydeepity/, examples/, or experiments/)

Platform(s) and compiler(s) tested:

Performance impact

None.

Checklist

  • I've read CONTRIBUTING.md, including the portability rules and numerical gotchas relevant to this change.
  • I've added a CHANGELOG.md entry under [Unreleased] (for any user-facing change).
  • I've updated relevant docs (README, Doxygen comments) if behavior or the public API changed.
  • New/changed code follows the project's .clang-format style.

Notes for reviewers

Also updated PyTorch's test_ffnn.py to use Sigmoid & ADAM. This is closer to the optimized MNIST test.

@Ra4ster
Ra4ster marked this pull request as ready for review September 3, 2026 03:34
@Ra4ster
Ra4ster merged commit bb927e8 into main Sep 3, 2026
11 of 12 checks passed
@Ra4ster
Ra4ster deleted the bindings/nanobindCI branch September 3, 2026 13:44
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.

1 participant