[perl] Odd number of elements in anonymous hash のエラー

Question

SVMLight 機械学習パッケージへの Perl インタフェースであるAlgorithm::SVMLightを使用し、SVMを構築しています。
http://www.mahoroba.ne.jp/~felix/Toolbox/Lang/Perl/Package/Algorithm-SVMLight.html
上記のサイトを参考にさせていただいております。

CSVファイルから属性とラベルを読み込み、学習インスタンスを追加していきたいのですが、
Odd number of elements in anonymous hashというエラーが出てしまいます。
<追記>－－－－－－－－－－－－
エラーは$s->add_instanceの部分で起きています。
$vectorfileから一行読み込むごとに一つずつOdd numberエラーが発生しています。
－－－－－－－－－－－－－－－－

調べてエラーの意味はわかったのですがどうすれば解決するのかがわかりません。
何か良い解決法はあるでしょうか。
ご教授いただけると嬉しいです。よろしくお願い致します。

vector.csvには全行同じ数の数値がカンマ区切りで格納されています。
label.csvにはvector.csvの行数と同じ数の数値がカンマ区切りで格納されています。

---

#!/usr/bin/perl
use strict;
use warnings;
use utf8;
use open IO => ":utf8";
use Encode qw/decode_utf8 encode_utf8/;
use Algorithm::SVMLight;

my $vectorfile = "vector.csv";
my $labelfile = "label.csv";

my $s = new Algorithm::SVMLight;

open(INLABEL,"<$labelfile"); #ファイル一括読み込み
local $\ = undef;
my $label = <INLABEL>;
close(INLABEL);

my @label = split(/,/,$label); #ラベルをカンマで区切る

my $j = 0;
open(INVECTOR,$vectorfile);
while(<INVECTOR>){
chomp;
my @vector = split(/,/, $_); #ベクトルをカンマで区切る

my $attribute = "";
for(my $i=0;$i<=$#vector;$i++){
    $attribute = $attribute . $i . "=>" . $vector[$i] . ",";
}
chop($attribute); #最後のカンマを消す

$s->add_instance
(attributes => {$attribute},label => $label[$j]);

$j++;

}

$s->train;

Comments

[argius]

$s->add_instanceのところで起きてるっぽいので、それも質問に書いておくと回答が得られやすくなると思います。（エラーにはat xxx.pl line 33のように出ていますよね？）

Answer 1

add_instanceメソッドの、attributesに設定する値の形式が誤っています。

ここは、文字列を設定するのではなく、ハッシュの参照を渡さなければなりません。

my %attrubutes = ();
for(my $i=0;$i<=$#vector;$i++){
    $attributes{$i} = $vector[$i];
}

$s->add_instance
    (attributes => \%attrubutes ,label => $label[$j]);

このように、ハッシュ変数に格納して、渡すときはリファレンスを渡します。

Comments

[mi-ko41]

解決しました。ありがとうございました！

Answer 2

実行環境もなく実際に試していませんし、SVMLight 機械学習パッケージも使用したことが無いので的外れかもしれませんが･･･

for(my $i=0;$i<=$#vector;$i++) {
   ･･･
}

これだとforループがvectorの要素数＋１回回ってしまってlabelの要素数より１つ多くなり、結果として
Odd number of elements in anonymous hash
が発生するのではないでしょうか。

Comments

[mi-ko41]

ご回答ありがとうございます。
$#vectorは最後の配列の要素番号になるので、間違ってはいないはずです。
試しに
for(my $i=0;$i<$#vector;$i++)
としてみましたが変わらずエラーが出てしまいました。