Skip to content

tensor engine, fix bugs to make test_remote_functions pass (#455) #466

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zdevito
Copy link
Contributor

@zdevito zdevito commented Jul 8, 2025

Summary:

Fixes:

  • formatting of error messages
  • transposition of all_gathers, via a workaround. the real fix relies on fixing nccl-comm actor to respect the order of dimensions in process groups.
  • cloning of WireValue, which drops the is_wrapped_number property incorrectly.
    ghstack-source-id: 294881245
    exported-using-ghexport

Differential Revision: D77880706

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 8, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77880706

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77880706

Summary:
Pull Request resolved: #469

Fixes:
* formatting of error messages
* transposition of all_gathers, via a workaround. the real fix relies on fixing nccl-comm actor to respect the order of dimensions in process groups.
* cloning of WireValue, which drops the is_wrapped_number property incorrectly.
ghstack-source-id: 295007765
exported-using-ghexport

Reviewed By: colin2328, suo

Differential Revision: D77880706
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77880706

@zdevito zdevito force-pushed the export-D77880706 branch from b214594 to 03f3696 Compare July 9, 2025 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants