Skip to content
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

One hot #2784

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

One hot #2784

wants to merge 3 commits into from

Conversation

akshitgaur2005
Copy link

Pull Request Template

Checklist

  • Confirmed that run-checks all script has been executed.
  • Made sure the book is up to date with changes in this PR.

Related Issues/PRs

#1544
#1714

Changes

Implement OneHot node for burn_import

Testing

Not yet tested.

This is a draft PR as I cannot figure out how to parse depth field of OneHot, as this is my first time working on such a project.

Any guidance will be helpful.

@laggui
Copy link
Member

laggui commented Feb 7, 2025

Btw in case you missed it we have a tensor.one_hot_fill(...) method that was added in #2613 and closely follows the ONNX spec 🙂

@akshitgaur2005
Copy link
Author

Oh, I missed it. I can't find its documentation though?

@laggui
Copy link
Member

laggui commented Feb 7, 2025

It was added after the last release, so it's not on the hosted documentation yet. But it's here (and in the book).

The up-to-date book on main can be browsed locally if you launch:

cargo xtask books burn open

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