Skip to content

Conversation

@NaderAlAwar
Copy link
Contributor

This checks if torch is installed and uses torch.is_tensor() to check if the passed argument is a pytorch tensor.

@NaderAlAwar NaderAlAwar mentioned this pull request Jul 7, 2024
@gliga
Copy link
Contributor

gliga commented Jul 9, 2024

If torch is not available, will it print a user friendly error message?

@NaderAlAwar
Copy link
Contributor Author

Not at the moment but this is not restricted to PyTorch. We need to have better error messages for invalid arguments passed to kernels.

Copy link
Contributor

@JBludau JBludau left a comment

Choose a reason for hiding this comment

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

the doctoring is wrong after the change

@IvanGrigorik IvanGrigorik requested review from JBludau and gliga and removed request for JBludau and gliga November 18, 2025 21:34
Copy link
Contributor

@gliga gliga left a comment

Choose a reason for hiding this comment

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

When are tests coming

@IvanGrigorik IvanGrigorik requested a review from JBludau November 20, 2025 04:19
@IvanGrigorik
Copy link
Collaborator

IvanGrigorik commented Nov 20, 2025

All good now, can't merge before @JBludau approval

Copy link
Contributor

@JBludau JBludau left a comment

Choose a reason for hiding this comment

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

looks good other than the file ending

@gliga
Copy link
Contributor

gliga commented Nov 20, 2025

As discussed in another PR, can we please add CI check for all line endings

@IvanGrigorik
Copy link
Collaborator

looks good other than the file ending

As discussed in another PR, can we please add CI check for all line endings

#315

@IvanGrigorik IvanGrigorik merged commit b42c805 into kokkos:main Nov 20, 2025
5 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.

4 participants