Skip to content

Conversation

@Eng-Abdelrahman-M
Copy link
Contributor

Added prod function

Have 3 errors in PyTorch
2 because of dtype PyTorch does not support uint16 , uint32 and uint64

  • input dtype = <undefined stub for 'uint16'> riase an invalid argument
  • input dtype = torch.uint8 requies expected = <undefined stub for 'uint32'>

1 error because of not applicable test case

  • input x = tensor([0], dtype=torch.uint8), axis = 1 raise Dimension out of range
    this error is right because of the tesor is 1 dim (0) there is no axis 1

Should I handle these errors or the problem in test cases?

@djl11 djl11 requested review from djl11 and removed request for djl11 March 14, 2022 10:58
@djl11 djl11 added the Pending External Bug Fix This issue or PR cannot be closed until an external bug is fixed label Mar 15, 2022
@djl11
Copy link

djl11 commented Mar 15, 2022

Have made a comment on an associated issue in the Array API test suite. Might be a small bug on their side.

@Eng-Abdelrahman-M
Copy link
Contributor Author

Thank you, Dan. I hope they fix it soon.

@Eng-Abdelrahman-M
Copy link
Contributor Author

Eng-Abdelrahman-M commented Mar 15, 2022

@djl11 Sorry Dan, The error you mentioned there can be fixed from our side by if statement. I already made if statement here.
The problem is if the function got an unsupported dtype. I think you already have fixed that one.
The two remaining problems are the axis problem, and how I can return uint32 by fixing the condition mentioned here.

@djl11 djl11 added Tests Pending and removed Pending External Bug Fix This issue or PR cannot be closed until an external bug is fixed labels Mar 17, 2022
@djl11 djl11 merged commit f4b78d3 into ivy-llc:master Mar 17, 2022
@djl11
Copy link

djl11 commented Mar 17, 2022

I have suppressed the unit test for now, and added a comment that this test is failing due to a bug with the Array API test suite. Thanks for the contribution!

@Eng-Abdelrahman-M
Copy link
Contributor Author

Thank you Dan!

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.

3 participants