.mean() on ArrayView1<u64>: panicked at 'attempt to add with overflow'

オープン
#880 コメント 12 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
42/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
rust
領域
data

調査の方向性

Start by running the provided Rust example and inspect the ndarray implementations of mean() and sum(), since the report identifies their likely interaction. Determine how mean should behave when summing u64 values beyond that type's range, then verify that the reproduction no longer panics and still returns the expected mean.

索引モデルが issue の本文から書いたものです。

説明

enhancement

I wanted to calculate a mean over an ArrayView1<u64> with 1000 values and encounter a panic:
thread 'main' panicked at 'attempt to add with overflow', /rustc/e1884a8e3c3e813aada8254edfa120e85bf5ffca/library/core/src/ops/arith.rs:107:1

Is this intended? Or should the u64 values internally be casted to u128 when calculating the mean?
The following code reproduces the error:

use ndarray::*;

fn main() {
    let mut data_vec: Vec<u64> = Vec::new();
    for _ in 1..1000 {
        data_vec.push(132542363533234561)
    }
    let data = data_vec.as_slice();
    let view: ArrayView1<u64> = ArrayView1::from_shape(data.len(), data).unwrap();
    println!("{:?}", view.mean());
}

I think the reason for this error is that mean() uses the public sum() function, which panics if the result does not fit into the element type. However, the mean result should always fit into the element type, so mean() maybe shouldn't use the public sum() function.

主要言語
Rust
スター
4.3k
フォーク
391
30日以内にマージされた PR はありません

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

rust-ndarray/ndarray のほかの issue

rust-ndarray/ndarray の issue をすべて見る

似ている issue

Rust の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。