Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
elのリミットの制限方法を修正しました。
もともといくつか挙動がおかしいところがあり、例えば
prev_angle[i] = m_qRef.data[i];
で1周期出力をセットしているのが最終出力になってなかった(このあとerrorのリミットで値が書き換えられる)などがあったので、修正しました。
具体的には、
(m_qRef - prev_angle)/dt < uvlimit
<=>m_qRef < uvlimit *dt + prev_angle
として角度のリミット形式にかきかえられる(m_qRef - m_qCurrent) < limit
<=>m_qRef < limit + m_qCurrent
として角度リミット形式にかきかえられるため、速度、角度、エラーの3点につきそれぞれ角度形式のリミットを計算して、
最後に
m_qRef=min(速度リミット、角度リミット、エラリミット)
のようにして3点のなかの一番厳しいリミット値で制限するようにしました。
また、速度・エラーなどもsamplerobot_softerror_imitter.pyでチェックして、多分travisのテストにもされると思います。
よろしくお願いいたします。