jenkins hashを使う #18

Merged
merged 2 commits into from May 1, 2017

Conversation

Projects
None yet
1 participant
@syou6162
Owner

syou6162 commented May 1, 2017

pprofで見ているとDotProdの計算時間が結構バカにならない感じになっていた。今は文字列がkeyのhashを何回も引いているので、そこをid化すればよさそう。単純にidに置き換えていってもいいけど、次元が決まっていたほうが後続の処理はやりやすいので、jenkins hashでやる。

@syou6162 syou6162 self-assigned this May 1, 2017

@syou6162

This comment has been minimized.

Show comment
Hide comment
@syou6162

syou6162 May 1, 2017

Owner

5:32.44くらいかかっていた処理が3:27.76で終わるようになって、精度はほぼ変わらずという感じなのでよさそう。次元が固定長になったので、重みベクトルはhashじゃなくてスライスでよさそうだけど、それは別PRでやろう。

Owner

syou6162 commented May 1, 2017

5:32.44くらいかかっていた処理が3:27.76で終わるようになって、精度はほぼ変わらずという感じなのでよさそう。次元が固定長になったので、重みベクトルはhashじゃなくてスライスでよさそうだけど、それは別PRでやろう。

+ return mod(hash, MaxFeatureLength)
+}
+
+func ExtractFeatures(state *State, pair ActionIndexPair) ([]int, error) {

This comment has been minimized.

@syou6162

syou6162 May 1, 2017

Owner

昔の名残で多値を返すようになっているけど、error吐くような場合は今ないので、変えてよさそう。

@syou6162

syou6162 May 1, 2017

Owner

昔の名残で多値を返すようになっているけど、error吐くような場合は今ないので、変えてよさそう。

@syou6162 syou6162 merged commit 3117bd9 into master May 1, 2017

1 check passed

ci/circleci Your tests passed on CircleCI!
Details

@syou6162 syou6162 deleted the jenkins_hash branch May 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment