RTMコンテスト2017作品感想(その2)

今日はこの作品の感想を書きます。

全体的な完成度が高く、今年の作品の中で1～2を争うぐらいの出来だと思います。
ただ何がアピールポイントなのか1番よくわからないのもこの作品です。
いったい今までと何が変わったのかをもう少し強調しないと辛い評価になるかもしれません。

起動スクリプトを用意しているのは親切だと思います。


ソースコードを読んだ感じですが、グローバル変数を多用しているのは気になります。
マジックナンバーも多いなあ。別にほかの作品も同じだけど。
ImageSelectorのonActivate関数でRTC_ERRORをそのまま書いてある箇所があるけど、これはおそらくreturnを付けるのが本来の意図なんだと思います。ほかにもミスはあるのでしょうけど、読む気力がないです。


データ型については人座標をTimedDoubleSeqにしているのは、独自データ型を使ったほうがよかったのではないかと思います。サービスポートをうまく使ったほうが、RTCの使い勝手がよくなる可能性はありそうです。RTCの設計についてはもっと考える余地があったのではないかと思います。






次はこの作品の感想を書きます。

マニュアルがちょっと雑だなあ。
そりゃページ数が多ければいいというわけではないけど、第三者が利用するための最低限の情報は必要です。
OpenRTM.NETを使った作品を出す人は珍しいです。

なんだか入出力データの単位がよくわかりません。
データポートで入出力するデータの単位はSI単位が基本です。
TimedVelocity2D型はm/s、rad/s、TimedPose2Dはm、radと決められています。
mmやcmでバラバラだと相互運用は不可能です。

それにしてもRSNPMobileRobotというRTCはどこで入手できるのだろうか。
あとデバッグビルドのファイル配布はやめてほしいです。

これだけマジックナンバーだらけなのは、少し問題だなあ。
onExecuteに300行以上も書いているのもまずいけど。









にほんブログ村のロボットのカテゴリから
全然人が来ない・・・