Skip to content

explicitly adding dim argument to torch.cross#23

Open
ventura-rivera wants to merge 1 commit intoRosettaCommons:mainfrom
ventura-rivera:patch-1
Open

explicitly adding dim argument to torch.cross#23
ventura-rivera wants to merge 1 commit intoRosettaCommons:mainfrom
ventura-rivera:patch-1

Conversation

@ventura-rivera
Copy link
Copy Markdown

There is a user warning of a deprecation for torch.cross. This tiny change fixes that to avoid unintended behavior in the future:

UserWarning: Using torch.cross without specifying the dim arg is deprecated.
Please either pass the dim explicitly or simply use torch.linalg.cross.
The default value of dim will change to agree with that of linalg.cross in a future release. (Triggered internally at ../aten/src/ATen/native/Cross.cpp:62.)

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.

1 participant