rok commented on issue #47520: URL: https://github.com/apache/arrow/issues/47520#issuecomment-3271087925
@andishgar thanks for finding and pointing these out! > In the above code, both positive and negative zero are considered zero; however, in the following code, negative zero is treated as a non-zero value. This is quite unfortunate and we should address it asap. @pitrou it would probably be best to fix this before the release. > @rok Is it okay to change the behavior of the following method to return a column-major tensor whenever necessary? (This refers to the second case mentioned in this https://github.com/apache/arrow/issues/47520#issue-3390224631.) I think that'd be ok, what kind of API would you propose? I imagine you wouldn't be satisfied with Tensor.to_column_major() that would make a data copy? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
