musyoku/vpylm

VPYLMのC++実装

vpylm's Issues

デバッグの助けになれば幸いです

src/vpylm.hの256行目、VPYLM::sample_next_token()の冒頭のsumの初期化のところですが、
double sum = pstop;
のように、pstopで初期化すべきな気がします。

また、
vpylmの木の各depthにbosに対応するtokenを登録すべきな気がします。
試しに、
model.cppの174行目でnew_depthを得たあと、
_vpylm->add_customer_at_timestep() を呼ぶ前に
if (new_depth==0) new_depth = 1;
とすると、多少改善すると思います。
depth=1のところにbosなnodeが追加されるからです。
勿論これはやっつけにしかすぎませんが。

もしトンチンカンなことを言っていたら済みません。