harujoh / KelpNet

Pure C# machine learning framework
Apache License 2.0
243 stars 28 forks source link

連続空白の扱いが怪しい? #17

Closed nyatla closed 7 years ago

nyatla commented 7 years ago

Splitの引数を省略すると、連続空白や制御文字+空白が空文字列として辞書に登録される場合がありました。

harujoh commented 7 years ago

ご指摘ありがとうございます。これ、完全にバグってますね・・・。

ただ、C#ではご提案いただいた空白付き引数でも、連続空白に対し空の配列を吐き出してしまうようです。 一旦マージさせていただき、更に対応をしたいと思います。

nyatla commented 7 years ago

ありがとうございます。 不完全なパッチですみません。

harujoh commented 7 years ago

いえいえ、見つけていただいただで十分ありがたいことです。

それに今回のケースは特にそうなんですが、コメントだけよりソースで示していただいたほうが、指摘の内容の意味を理解する事ができるので、ソースで示して頂けるのは助かります。