忍者ブログ
ロボット、千葉ロッテマリーンズについていいかげんなことを書きます。
[3]  [4]  [5]  [6]  [7]  [8]  [9]  [10]  [11]  [12]  [13
×

[PR]上記の広告は3ヶ月以上新規記事投稿のないブログに表示されています。新しい記事を書く事で広告が消えます。

勘違いかと思って確認してみたのですが、やはり18作品でした

でもあの後新たな作品投稿どころか全く更新自体がないみたいですが、どうしたのでしょうね?



新たにソースコードを公開した作品もないので、今日は2作品についてしか感想が書けません。


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

デバッグビルドしているのでリリースビルドして配布してほしいです。

マニュアルを読んでみたのですが、情報が不足しすぎです。
どっちかと言えば概要に近いような感じです。

気になったのが単位にmmを使っている事です。
基本的にはmを使ってほしいです。コンフィギュレーションパラメータはmmにしないと直感的でないかもしれないので、これは仕方ないと思います。
ただ、データポートはmでやってほしいです。


ソースコードを読んだ感想を書きます。

Humanと言う構造体で各配列の数を100に設定していますが、これはstd::vectorを使った方がよいかもしれないです。
onExecute関数でm_SL.data.length等が100を超えるとアクセス違反で落ちるかもしれないです。

全体的に力技と言うか、無理やりなコードの書き方のように感じました。もう少し工夫があったらよかったかもしれません。頑張っているとは思うのですけど。
Human_Step.cppなんかはstd::vector等コンテナを使えば簡単に記述できそうな所が多いように思います。


どうでもいいのですけど、円周率はmath.hのM_PIを使えばいいのではないでしょうか?
#define _USE_MATH_DEFINESをmath.hをインクルードする前に記述すれば使用できます。






次のこの作品の感想を書きます。
・・・何と言うか、突っ込みどころが満載すぎてどこから手を付けていいのか分かりません。


まずプロジェクトページからしてかなり雑です。
マニュアルがやっつけすぎです。
せめてRTCの仕様ぐらいは書いてほしいです。


ソースコードを読んでみた感想を書きます。
まずはPCANBasic.hとPCANBasicCLR.hは誰が作成したのでしょうか?
昨日も同じことを言ったような気がしますが、第三者から見たらこのコードの作者が作品に関係あるのかないのかは判断できません。ソースコードの審査があるのでこういう事はやめてほしいです。
仮に関係ない人物だとして、このソースコードのライセンスは何でしょうか?
そもそも再配布できるライセンスなのかどうかが不明です。
しかも作品のソースコードの半分を占めるぐらいの量なので、見つかったら怒られるどころでは済まないかもしれないです。
人のソースコードを使わせてもらっているのにこれは酷いと思いました。

PCANBasic.dllが必要みたいですが、これはどこから入手するのでしょうか?
おそらくPCANBasic.hやPCANBasicCLR.hと同じ場所から入手できるとは思うので、それならPCANBasic.hやPCANBasicCLR.hも除外しておいた方がいいと思います。

それからRoboCar.cppでrobocarDll.hというファイルをインクルードしていますが、このファイルはどこにもないようなのでコンパイル自体失敗すると思います。

気になったのがfpというファイルポインタですが、これがどこでも初期化されていないのにonDeactivatedでfclose関数が呼び出されています。ちょっと意味が分かりません。




ソースコードやマニュアルよりも、まずは著作権に問題があるのかないのかを確認してほしいです。




早速暗雲が立ち込めてきたというか、危惧していた事が現実味を帯びてきた感じです。
ここまで見た感じでも著作権的に問題があるかもしれない作品が4つもあったので注意喚起ぐらいはした方がよいのではないでしょうか?
大きなトラブルに発展する可能性があるのはこの作品だけだとは思いますけど、念のためです。






僕の勝手な考えですけど、マニュアルには最低限以下の情報が必要だと思います。
  1. 概要
  2. RTCの仕様
  3. システムの構成図
  4. 必要なソフトウェアのリスト
  5. 必要なハードウェアのリスト
  6. インストールの手順
  7. 使用方法
  8. (必要であれば)チュートリアル
  9. 使用した、参考にしたソースコード、資料のリスト
  10. ライセンス
まあ意見は分かれるところだとは思いますけど、第三者が再現できるぐらいの情報はほしいです。










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

人気ブログランキングへ
PR
流石に10作品以上の感想を一気に書くのは辛いので3作品だけ書きます。


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

マニュアルを読んでみたのですが、全体的に不親切な印象です。
まずどうやってRTCを使用してほしいのかが分かりませんでしたし、どういうシステムを作成したのかについての説明がないので内容自体がよく分かりません。
どうやって使ってほしいのか?という部分の説明が不足しています。第三者がシステムを再現するために必要最低限の情報が足りていません。
もっとマニュアルを加筆しないと厳しいかなという印象でした。



ソースコードを読んだ感想を書きます。
SixLevelColourMapper.hでcolourという変数が定義されていますが、どうやら値が変わることはないようなので定数でいいと思います。

public:
    static const RTC::RGBColour colour[6];

const RTC::RGBColour sample1::colour[6] = {(省略)}
こんな感じで。


あとHREstimator.cppでWindows APIのSleep関数を使っていますが、移植性を考えたらcoilの関数を使った方がよいかもしれません。それ以前にonExecute関数内でsleep関数を使う意図はなんでしょうか?
HREstimatorというRTCからセンサーの値を好きな時に取得する事はできないわけで、しかもonExexute自体が実行コンテキストで周期的に駆動しているので、sampleTimeというコンフィギュレーションパラメータで設定した時間ごとに計測はできていないと思います。
本当に正しい心拍数が計算できているのでしょうか?








次にこの作品の感想を書きます。
OpenRTM-aistの1.1.1ではなく1.1.0を使っている事もまず疑問ですが、どうしてデバッグビルドしているのでしょうか?

マニュアルを読んでみたのですが、このマニュアルを読んでシステムを再現するのは無理だと思いました。情報が足らなすぎます。


よく分かりませんけど、写真の日付は勝手に入れてくれる方が便利ではないでしょうか?
コンフィギュレーションパラメータでいちいち設定し直すのは面倒です。
コメントも音声などで入力できるようにしておいた方が面白そうです。



ソースコードを読んでみた感想を書きます。
まず、cvCustom.hというファイルは誰が作成したのでしょうか?
第三者から見たらこのコードに上部に記載されている人物がこの作品に関係あるのかないのかは判別できないわけで、ソースコードでの審査があるわけですからこういうややこしいことはやめてほしいです。
関係ある人物ならば構いませんが、関係ない人物の場合はマニュアルに使用を明記した方が安全です。
まあこの程度の量なら怒る人もあまりいないでしょうけど、無用なトラブルを避けるためです。
安全とか危険とか以前に、このコードの作者への最低限の敬意は払うようにしてください。




ちょっとマジックナンバーが多いかなとは思いました。
定数で定義しておいて使った方がよいかもしれません。







最後にこの作品の感想を書きます。
まずOpenRTM-asistってなんでしょうか?
マニュアルでもこう書いてあるので誤字ではなく勘違いをしている可能性がありそうです。

CropImageのCroners等は独自のデータ型を使った方が分かりやすいかもしれないです。
まあ意見の分かれるところでしょうけど。

SysMLの図を付属してるというのを見て少し不安になりました。変な実装になっていなければいいのですが。
内部ブロック図を見た感じでは多分大丈夫かな?と言う感じはします。

ただ実際のRTシステムを見た感じ、これはよくないなと思いました。
LongSeqJointer、LongSeqSplitter、IR_Transceiverを分割しているのは一気にシステムの見通しを悪くしています。
おそらくIR_TransceiverとLongSeqJointer、LongSeqSplitterは他のRTCには置き換える事の出来ない不可分な関係になっているので、分割する意味はありません。
確かにこの手の処理はArduino言語では難しいので気持ちは分からなくはないですが、何とかならなかったのかなとは思います。
どうやらRaspberry PIを使う場合は単一のRTCで大丈夫みたいなのでいいのですけど、Arduinoを使う場合は少し気になります。

IRControllerとLED_Detectorを分割する必要はあったのかな?と思いました。
よく分かりませんが、この2つのRTCは他の用途で使用可能なのでしょうか?
一緒でも良かったかもしれませんが、LED_Detectorは他に接続可能なRTCも作成できるかもしれないので何とも言えません。



マニュアルを読んでみた感じですが、動作させるのは無理そうです。
自動起動のスクリプトを作ってほしいです。
起動してからの使い方もよく分かりません。動画だけではなくマニュアルでも説明してほしいです。




ソースコードを読んでみた感じ、かなり頑張っているように思いました。
参考になる部分も多かったように思います。

WebUI.pyですが、onExecuteでデータの送受信を処理する必要があるのかな?と思いました。
どうやらonExecuteでInImageのデータを読み込んでグローバル変数g_imgに圧縮した画像を格納しているみたいです。そしてWSHandlerのloop内でg_imgから画像を描画するようにしているみたいですが、それならデータの読み込みもloop内で処理すればよかったのではないでしょうか?
逆にインスタンス化したRTCをグローバル変数に格納してWSHandlerのloop内で使用すれば可能だと思いますけど。















数えてみたところ13作品しか登録されていません
18だったと思うのですが、ひょっとしたら勘違いかもしれないです。














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

人気ブログランキングへ

次はこの作品の感想を書きます。
今のところ一番の難敵になりそうな出来ですね。




マニュアルを読んでみた感じで、どうしてOpenRTM-aistの1.1.0を使っているのかに引っかかりました。1.1.1が出ているのですからそっちを使えばいいと思います。
後は実行ファイルやDLLのファイル名が全て小文字になっている事からRTCビルダも古いバージョンを使用しているみたいです。

それからDSVL.dllやlibARvideo.dllを付けるのだったらOpenRTM-aistやomniORB関連のDLLも付属してくれたら良かったと思います。


やっぱりpdfのマニュアルは読みづらいです。
それに使う側としてはマニュアルなどを読んで内容を知ってからソフトウェア本体を入手したいので別々にダウンロードできるようにしてほしいです。
マニュアルを作る側もちょっと修正するだけで全体をアップロードし直すのは手間だと思います。


DetectArMarkerRTC_解説マニュアルやVS_ASR_RTC_解説マニュアルでプロジェクト生成後にインクルードパス等を編集するように指示していますが、CMakeLists.txtに記述しておいてほしいです。




ここからはソースコードを読んでみた感想を書きます。

ちょっとグローバル変数を使いすぎです。
何と言うかこの人は個性的と言うか、独特なコーディングをしますね。
ここまで徹底的にグローバル関数を多用するC++のプログラムは滅多に見ないと思います。


useSilabs.hのdevというグローバル変数を定義しているのに、RSTorqueOnOff等の関数で同名のdevという引数を使うのは混乱します。
それに以前から言っている通りグローバル変数や静的メンバ変数を使うとrtcdで起動するときに困ります。
スカラロボットが複数接続されている場合に必ず1台目のロボットに接続するようになっているみたいですが、これはコンフィギュレーションパラメータで設定できるようにしておいたほうが良かったかもしれません。

あとは、順運動学とか逆運動学を何度も使うのであれば関数にまとめて使いまわした方が便利だと思います。

実装の都合上仕方ない部分もあるとは思いますが、キー入力などでonExecute等の処理を止めてしまうのは好ましくありません。





useSilabs.cppで定義されている関数は見た感じスカラロボットのサンプルプログラムと同じようですが、大丈夫でしょうか?
許可を貰ったとか書いてあったような気がするので多分大丈夫だとは思いますけど、使用が明記されていないのでなんだか気になります。



MITライセンスとなっていますが、ARToolKitを使ってビルドした実行ファイルを配布している時点でGPL感染しているのではないでしょうか?

どうにもライセンスが怪しいので確認の必要があるかもしれないです。







他の作品の感想を書きたいのですが、まだソースコードが公開されていないようです。
あと7時間弱で締め切りですが、去年もこんな感じだったので大抵の人は間に合います。
と言うわけで、明日は10作品以上の感想を書くとは思います。





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

人気ブログランキングへ
色々と作品が投稿されているので感想を書きたいのですが、その前に結局盗用の件がどうなったのか分からないので知っている人は教えてください。

何の動きもなさそうな所を見ると、おそらく本人とコンテスト運営側で話はついているのだとは思うのでもう触れない方がよいのかもしれません。
賞の剥奪がないと言う事は審査にミスは無かったという事だと思うので、全然納得はできませんがこの件は完全に鎮火した問題と言う事だと思います。

去年のRTMコンテストの盗用の件は大事にならずに済んだみたいですが、最悪の場合は逮捕者が出ていたかもしれません。なので著作権には細心の注意を払うようにしてください。


流石にもうこの件には二度と触れないようにします。











それはさておき今年の作品の感想を書きます。

まずはこの作品の感想から書きます。
発想が面白いと思いましたし、Pythonで実装しているのでRTC自体は動作はできそうです。
でもsample.pdfを読んだ印象で、第三者がシステムを動作させるのは難しいなと思いました。
よく分かりませんが各RTCを手動で起動して、さらにデータポートを10箇所以上も接続して、さらにコンフィギュレーションパラメータも設定してほしいという事でしょうか?
流石にそれは無茶です。

今思い出したのですが、この質問をした人と関係がありそうですね。
と言う事はこの質問は誰も答えずに放置されていたみたいです。
誰か返信するか、僕にbeginnersのメーリングリストの登録の仕方を教えてください。


この作品のマニュアルの最大の問題はシステムの全体図が一切ない所かもしれません。
突然Raspberry Piとか出されてもどこで使ったのかが分かりませんでした。普通のPCをRaspberry Piに置き換えることもできるよと言う事でしょうか?

肝心のXMLファイルの書き方の説明がどこにもありません。
サンプルだけ試して終わりだったら使う側としてはあまり旨味がないように思います。


RTCを細分化しすぎな感じはありました。
XMLloaderとXMLCentralControllerRTCを分けた意図が分かりませんでした。
XMLloaderはどうやらファイルを読み込んで文字列を出力しているだけのようなのでわざわざ分割するまでもないと思います。
それからXMLDelayerRTCも分ける必要があったか疑問です。
XMLCentralControllerRTCと一緒のRTCにして、使いたくない時はXMLファイルでdelayタグを使わなければいいだけではないでしょうか?


漫才自体が微妙でした。せっかく笑いを取りに来たのであればぶっ飛んだ事をしないと見ている方もどう反応をしていいのか困ります。ロボットがいきなり爆発してもよかったぐらいです。
せめて表示される顔を卑猥なものにするとか工夫があった方がよかったと思います。



ここからはソースコードを読んでみた感想を書きます。

BeautoRoverRTC.pyでシリアルポートのオープンに失敗したときはエラーに遷移する方がいいかなとは思いました。

ものすごいどうでもいい事ですが、PythonでNoneと比較するときはif A is None:とisを使うのが推奨されています。まあどうでもいいですけど。







どうやら他にソースコードが公開されている作品はないようなので、他の作品の感想は明日書きます。










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

人気ブログランキングへ
あれから公開された作品もないし書くことがありません。
締め切りまで1週間しかないわけですが、締め切り直前にまとめて公開されるとは思います。
今年は去年よりは参加者も少ないのでソースコードやマニュアルを読むのも楽そうです。




どうやら前回の記事は作者様が見てしまったらしいですね。




「目つけられた」とか人聞きの悪いこと言うなよとか思わなくもないですが、僕の方が力で劣るのだから他の人の作品を研究するのは当たり前です。
いいところがあったら当然パクりますし、悪いところがあれば自分はやっていないか入念に確認します。




とりあえず僕もプロジェクトページを作ってみました。


複合コンポーネント作成支援ツール
4自由度ロボットアーム制御RTコンポーネント群


例のごとく自分で5点の評価を付けています。
評価が高い方が上に表示されやすいみたいなので仕方ないです。
文句のある人はログインして正当な評価を付けてください。





何故かファイルの添付ができませんでした。
「割り当てディスク容量の 1024 MB を超過しています。」と出ますけど、今までアップロードしたファイルの容量を通算しても200MBもないと思います。
確信はないのですが、一度アップロードしたファイルは削除しても内部的には削除したことになっていないみたいです。少し困りました。

とりあえずは別の場所にインストーラーをアップロードしておきます。













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

人気ブログランキングへ
カレンダー
03 2024/04 05
S M T W T F S
1 2 3 4 5 6
7 8 9 10 11 12 13
14 15 16 17 18 19 20
21 22 23 24 25 26 27
28 29 30
フリーエリア
最新CM
[08/31 ysuga]
[08/31 Nobu]
[08/31 ysuga]
[12/11 Nobu]
[12/11 Kanamura]
最新TB
プロフィール
HN:
Nobu
年齢:
35
性別:
男性
誕生日:
1988/09/22
職業:
あれ
趣味:
妄想、自堕落
バーコード
ブログ内検索
P R
カウンター
忍者ブログ [PR]