rshinden / training_guide_RTC

training_guide_RTC(contest)
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

RTCを疎結合にした方がいい #3

Open Nobu19800 opened 4 years ago

Nobu19800 commented 4 years ago

個々のRTCが他のRTCへ依存しすぎていると思います。 例えばJudgeRTCとSerialConnectのデータポートを接続していますが、このSerialConnectは他のRTCに交換するのは困難です。 SerialConnectは実質的に圧力センサの値を取り出すRTCですが、別の圧力センサを使うと同じ力を加えても出力する値が違うので動作が変わってしまいます。 judge.pyには990や12等のSerialConnectと圧力センサに依存した値を直接記述しているため、RTCの交換が難しくなっています。 設計方針を見直した方がいいと思います。

rshinden commented 4 years ago

ご指摘ありがとうございます。 今までは、SerialConnectRTCからアナログ値が文字列として送られ、それぞれのRTCで数値の荷重に校正していました。990などは校正式の値です。そのため、新たにセンサ値を校正するChangeRTCを作成し、それぞれのRTCには初めから数値の荷重が送られるように変更致しました。 また、12は立ち上がり判定を行う閾値になっていますので、コンフィグレーションパラメーターにし変更できるように致しました。 詳しくはuser_manual_v1.5をご参照ください。 RTCの疎結合につきましては、今後設計方針を見直します。