teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

追加

2017/11/05 04:34

投稿

WathMorks
WathMorks

スコア1582

answer CHANGED
@@ -2,7 +2,8 @@
2
2
  ラベル部分を取り除いて画像データだけを抽出するためにcols.pop(0)が使われています。
3
3
  labels.append(int(cols.pop(0)))#<-----cols[0]を取り出してlabelsに追加し、cols[0]を削除している。
4
4
 
5
+ 可読性を考えると、ラベル・画像の位置を明示的に与え、colsそのものに変更を加えないほうがよいでしょう。
6
+ 例:
5
7
  labels.append(int(cols[0]))
6
8
  vals = list(map(lambda n: int(n) / 256, cols[1::]))
7
- images.append(vals)
9
+ images.append(vals)
8
- のように、明示的に書く方がわかりやすいと思います。

2

追加

2017/11/05 04:34

投稿

WathMorks
WathMorks

スコア1582

answer CHANGED
@@ -1,7 +1,8 @@
1
1
  各行の1番目(cols[0])はラベル、それ以降は画像データです。
2
2
  ラベル部分を取り除いて画像データだけを抽出するためにcols.pop(0)が使われています。
3
+ labels.append(int(cols.pop(0)))#<-----cols[0]を取り出してlabelsに追加し、cols[0]を削除している。
3
4
 
4
- labels.append(int(cols[0]))#<-----cols[0]を取り出してlabelsに追加し、cols[0]を削除している。
5
+ labels.append(int(cols[0]))
5
6
  vals = list(map(lambda n: int(n) / 256, cols[1::]))
6
7
  images.append(vals)
7
8
  のように、明示的に書く方がわかりやすいと思います。

1

追加

2017/11/05 04:31

投稿

WathMorks
WathMorks

スコア1582

answer CHANGED
@@ -1,7 +1,7 @@
1
1
  各行の1番目(cols[0])はラベル、それ以降は画像データです。
2
2
  ラベル部分を取り除いて画像データだけを抽出するためにcols.pop(0)が使われています。
3
3
 
4
- labels.append(int(cols[0]))
4
+ labels.append(int(cols[0]))#<-----cols[0]を取り出してlabelsに追加し、cols[0]を削除している。
5
5
  vals = list(map(lambda n: int(n) / 256, cols[1::]))
6
6
  images.append(vals)
7
7
  のように、明示的に書く方がわかりやすいと思います。