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

Add beep sound for CollisionDetector #476

Closed
wants to merge 1 commit into from

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Feb 10, 2015

CollisionDetectorで干渉時にビープ音がなるようにしました。
音は1秒間に
 ピッピッ(休み)
を繰り返します(SoftErrroLimiterなども含めて、document?が必要なきがしました)。

よろしくお願いします。

@k-okada
Copy link
Contributor

k-okada commented Feb 10, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1754/

Build Log
last 10 lines

[...truncated 5356 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:199)
    at hudson.model.Run.execute(Run.java:1769)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:89)
    at hudson.model.Executor.run(Executor.java:240)

Test FAILed.

@k-okada
Copy link
Contributor

k-okada commented Feb 10, 2015

some function is not found on QNX, check if they include Hrpsys.h https://github.com/fkanehiro/hrpsys-base/blob/master/lib/util/Hrpsys.h#L10

:Vector3&, hrp::Vector3&, const hrp::Vector3&, const hrp::Vector3&)':
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/RPYKalmanFilter.h:82: error: 'sqrt' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/RPYKalmanFilter.h:85: error: 'atan2' was not declared in this scope
In file included from /tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:13:
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/EKFilter.h: In member function 'Eigen::Matrix<double, 3, 1, 0, 3, 1> EKFilter::accelerationToRpy(const double&, const double&, const double&)':
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/EKFilter.h:31: error: 'atan2' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/EKFilter.h:32: error: 'sqrt' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp: In function 'int main(int, char**)':
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:32: error: 'atof' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:34: error: 'atof' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:36: error: 'atof' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:38: error: 'atof' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:82: error: 'FILE' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:82: error: 'gp' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:84: error: 'size_t' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:84: error: expected ';' before 'ii'
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:84: error: 'ii' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:86: error: 'fprintf' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:91: error: 'fflush' was not declared in this scope
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:96: error: expected ';' before 'j'
/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/testKalmanFilterEstimation.cpp:96: error: 'j' was not declared in this scope

@snozawa
Copy link
Contributor Author

snozawa commented Feb 11, 2015

@k-okada さん
上記のように,QNXでのエラーメッセージがでてるようです.
jenkins内でのビルドでエラーがでてるということでしょうか.

該当箇所の例えば

/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/RPYKalmanFilter.h:82: error: 'sqrt' was not declared in this scope

の箇所は,最近ファイルを分割したくらいで,もともとsqrt関数はだいぶまえからのKalmanFilterにはいっており,
特にこのようなエラーがでてなかったきがします.
どうなおしたらよいでしょうか.

@k-okada
Copy link
Contributor

k-okada commented Feb 11, 2015

https://github.com/fkanehiro/hrpsys-base/blob/master/rtc/KalmanFilter/RPYKalmanFilter.h#L4
あたりでinclude "util/Hrpsys.h"とかしないとだめなんじゃないかな.

◉ Kei Okada

2015-02-11 22:18 GMT+09:00 Shunichi Nozawa notifications@github.com:

@k-okada https://github.com/k-okada さん
上記のように,QNXでのエラーメッセージがでてるようです.
jenkins内でのビルドでエラーがでてるということでしょうか.

該当箇所の例えば

/tmp/hrpsys-source-476-2015-02-11/src/hrpsys-base-476/rtc/KalmanFilter/RPYKalmanFilter.h:82: error: 'sqrt' was not declared in this scope

の箇所は,最近ファイルを分割したくらいで,もともとsqrt関数はだいぶまえからのKalmanFilterにはいっており,
特にこのようなエラーがでてなかったきがします.
どうなおしたらよいでしょうか.


Reply to this email directly or view it on GitHub
#476 (comment).

snozawa added a commit to snozawa/hrpsys-base that referenced this pull request Feb 11, 2015
@snozawa
Copy link
Contributor Author

snozawa commented Feb 11, 2015

ありがとうございます、
#478
で修正してみました。

@k-okada k-okada mentioned this pull request Feb 22, 2015
@k-okada
Copy link
Contributor

k-okada commented Feb 22, 2015

k-okada@58b0293

みたいなのが必要かな.

@snozawa
Copy link
Contributor Author

snozawa commented Feb 26, 2015

#498
で対応できましたので、closeします。
ありがとうございました。

@snozawa snozawa closed this Feb 26, 2015
@snozawa snozawa deleted the add_beep_sound_for_co branch February 26, 2015 16:43
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.

None yet

2 participants