Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Compatibility with jit.script and torch.compile: COMPLETE #40

Merged
merged 55 commits into from
Dec 9, 2024

Conversation

borisfom
Copy link
Collaborator

@borisfom borisfom commented Dec 4, 2024

No description provided.

@borisfom borisfom requested a review from mariogeiger December 4, 2024 22:06
@borisfom
Copy link
Collaborator Author

borisfom commented Dec 4, 2024

Note: depends on backend MR #74.

borisfom and others added 5 commits December 4, 2024 20:12
Signed-off-by: Boris Fomitchev <bfomitchev@nvidia.com>
Signed-off-by: Boris Fomitchev <bfomitchev@nvidia.com>
…t-script

Signed-off-by: Boris Fomitchev <bfomitchev@nvidia.com>
@borisfom borisfom requested a review from mariogeiger December 6, 2024 01:22
@mariogeiger mariogeiger requested a review from stadlmax December 6, 2024 11:07
Copy link
Collaborator

@mariogeiger mariogeiger left a comment

Choose a reason for hiding this comment

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

I think we can merge

Comment on lines -33 to +35
@pytest.mark.parametrize("batch", [0, 32])
@pytest.mark.parametrize("batch", [1, 32])
Copy link
Collaborator

Choose a reason for hiding this comment

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

We must not forget to add back this test batch=0 when the backend properly supports it

@mariogeiger mariogeiger merged commit 3557c31 into NVIDIA:main Dec 9, 2024
11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants