core-rocket / amatsukaze

2019年11月大島
2 stars 1 forks source link

反省・感想(メイン電装) #33

Open sksat opened 4 years ago

sksat commented 4 years ago

良かったとこ

悪かったとこ

sksat commented 4 years ago

https://github.com/core-rocket/amatsukaze/blob/70da9a7d5914c9a1c1deeedb65292c4230dce4df/Avionics/src/main.cpp#L123 これまさかMPUのinitializeだけでレンジの設定してなくないですか?????

sksat commented 4 years ago

これだと±2gです......xyzどれかと機体軸が大体同じになってたことを踏まえるとこれは離床判定絶対にされない運命だったということですね............レビューした時に気付くべきでした.........

https://github.com/jrowberg/i2cdevlib/blob/1075cf7bcb88e4f2721f33a92233873141357c1c/Arduino/MPU6050/MPU6050.cpp#L59

sksat commented 4 years ago

CORE内で判定の実装のチェックリストみたいなのを作りましょう,作ります

Yukuro commented 4 years ago

良かったところ

悪かったところ

これまさかMPUのinitializeだけでレンジの設定してなくないですか?????

  • してませんでしたしてました...
sksat commented 4 years ago

レンジ設定どこでしてますか?

Yukuro commented 4 years ago

上の投稿は誤記でしてませんでした...(修正済み)