klion26 opened a new pull request, #8041: URL: https://github.com/apache/arrow-rs/pull/8041
# Which issue does this PR close? - Closes #8013 . # What changes are included in this PR? Improve the logic `i256` to `f64` by 1. Erase all the leading sign bit in `i256`, this will right-pad 0, 2. Treat the convert logic as `i256` = `the f64 descrbied as the left 64 bit after step 1` * `the scale( = pow(2, 192 - the bit count we erased in the first step)` -- the fraction bits in f64 is 52, i64 is enought for it. 2.1 Convert the left 64 bits in the first step to f64 2.2 Multiply the scale (`pow(2, 192 - the bit count we erased in the first step)`) # Are these changes tested? Covered by the existing tests, and add some more tests # Are there any user-facing changes? No -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
