We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The current implementation of the Tensor::log_sum_exp() function does not handle small numbers correctly. For example, the following code
Tensor::log_sum_exp()
let arr = Tensor::full(-1000_f64, (10,), &Device::Cpu)?; let logsumexp = arr.log_sum_exp(0)?; println!("{:?}", logsumexp);
prints
Tensor[-inf; f64]
whereas on torch 2.3.0
>>> arr = torch.tensor([-1000]*10) >>> torch.logsumexp(arr, 0) tensor(-997.6974)
candle-core = "0.6.0" Tested on Arch Linux
The text was updated successfully, but these errors were encountered:
Closing based on the fix in #2367, feel free to reopen if it's still an issue.
Sorry, something went wrong.
No branches or pull requests
The current implementation of the
Tensor::log_sum_exp()
function does not handle small numbers correctly. For example, the following codeprints
whereas on torch 2.3.0
Version info
candle-core = "0.6.0"
Tested on Arch Linux
The text was updated successfully, but these errors were encountered: