Skip to content

Fix MPS device movement for constant tensors in TabNet modules#278

Merged
DanielAvdar merged 3 commits intomainfrom
add-simple-tests-tab-network
Nov 2, 2025
Merged

Fix MPS device movement for constant tensors in TabNet modules#278
DanielAvdar merged 3 commits intomainfrom
add-simple-tests-tab-network

Conversation

@DanielAvdar
Copy link
Owner

No description provided.

Copilot AI and others added 3 commits November 2, 2025 13:22
Co-authored-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
* Initial plan

* Fix MPS device movement issues by registering tensors as buffers

Co-authored-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>

* Final verification: all tests pass, no security issues

Co-authored-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>

* Remove spurious nul file

* Simplify device selection logic

Removed MPS device check from auto device selection.

Signed-off-by: Daniel Avdar <66269169+DanielAvdar@users.noreply.github.com>

* Fix device selection logic for 'auto' case

Signed-off-by: Daniel Avdar <66269169+DanielAvdar@users.noreply.github.com>

---------

Signed-off-by: Daniel Avdar <66269169+DanielAvdar@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>
@DanielAvdar DanielAvdar merged commit dfc0b33 into main Nov 2, 2025
24 checks passed
@DanielAvdar DanielAvdar changed the title Add simple tests tab network Fix MPS device movement for constant tensors in TabNet modules Nov 2, 2025
@DanielAvdar DanielAvdar deleted the add-simple-tests-tab-network branch November 2, 2025 16:57
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

Comments