Skip to content

Conversation

@ncguilbeault
Copy link
Collaborator

Overview

This PR adds a few new operators and enhancements to the Bonsai.ML.Torch package. Major additions include new operators for buffering, deconstructing, and printing tensors, as well as operators for controlling gradient and inference modes. There are also improvements to file format support and device initialization, plus a new base class for automatic tensor type conversion.

…n is only happening on subscription and not before that
…` to avoid confusion with what is considered "native" in this case (is .NET native or torch native?)
…ors true function (splitting along dimension)
@ncguilbeault ncguilbeault force-pushed the dev/torch-core-additions branch from c7c4771 to 26900fd Compare November 11, 2025 14:27
Copy link
Member

@glopesdev glopesdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ncguilbeault Looks good to me, I just made comments on a few missing XML docs in case you want to fill them in, otherwise I am happy to merge this.

@glopesdev
Copy link
Member

@ncguilbeault All looks good to me, I would just look into the few remaining unresolved conversations above, especially the ones around tensor.save and tensor.load to remove the use of native torch methods for the time being, and then I feel we should be good to go.

Copy link
Member

@glopesdev glopesdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks great now, just a few missing bits of documentation, but we could also leave them for the next PR, if easier.

Copy link
Member

@glopesdev glopesdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed there are also some missing XML doc strings in other packages, and this was already flagged in #54 anyway, so I suggest we do a separate PR later to resolve this. For now I'm happy to merge this branch as-is.

@glopesdev glopesdev merged commit f4ea418 into bonsai-rx:main Dec 19, 2025
10 checks passed
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.

2 participants