Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update beep sound of SoftErrorLimitter #403

Merged
merged 1 commit into from
Nov 12, 2014

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Nov 11, 2014

SoftErrorLimiterが現在ビープ音を出していますが、
他のRTCなどとかぶらないように音を分けるようにしました。
[1] の議論では音階等をかえる案もでておりましたが、聞き分けやすい(と個人的に思いましたので)
音のリズムをかえるようにしました。
soft error 時には1秒に4回ピッピッピッピッとなり、position error 時には1秒に2回ピーッピーッとなります。

また、ドキュメントにも音とこのRTCの挙動について、説明を追加しました。

[1] #220

StableRTCですが、APIはかわらず、音のならし方のみかわる変更になりますが、よろしいでしょうか。

… confusing alert sound. This commit is discussed in fkanehiro#220
@k-okada
Copy link
Contributor

k-okada commented Nov 11, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1692/

@k-okada
Copy link
Contributor

k-okada commented Nov 11, 2014

StableRTCですが、APIはかわらず、音のならし方のみかわる変更になりますが、よろしいでしょうか。

はい.挙動とAPIが変わらなければ良いと思います.

@snozawa
Copy link
Contributor Author

snozawa commented Nov 11, 2014

はい.挙動とAPIが変わらなければ良いと思います.

はい、関節上書きの挙動・APIかわらない変更となります。

@fkanehiro さん、お手数ですが、travis restartの末mergeいただけますと助かります。

fkanehiro added a commit that referenced this pull request Nov 12, 2014
Update beep sound of SoftErrorLimitter
@fkanehiro fkanehiro merged commit ff6816b into fkanehiro:master Nov 12, 2014
@snozawa snozawa deleted the fix_beep_sound_of_el branch November 18, 2014 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants