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 qpOases for Stabilizer #683

Merged
merged 3 commits into from
Jun 29, 2015
Merged

Add qpOases for Stabilizer #683

merged 3 commits into from
Jun 29, 2015

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Jun 12, 2015

Stabilizerの計算で使っているQPライブラリを追加したいと考えてます。
qpOasesというライブラリでLGPLです。

@fkanehiroさん、@k-okadaさん、
CollisionDetectorのしたのvclip同様ソースをいれてみていますが、自身ないです。
外部ライブラリを追加するときにどうすべき、という指針がございましたらご教示いただけますと幸いです。

@k-okadaさん
QNXで動作確認していないのですが、CMakeの中で
if(QNX)のようなものはかくことはできるのでしょうか。

よろしくお願いいたします。

@k-okada
Copy link
Contributor

k-okada commented Jun 12, 2015

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

@snozawa snozawa closed this Jun 12, 2015
@snozawa snozawa reopened this Jun 12, 2015
@snozawa
Copy link
Contributor Author

snozawa commented Jun 12, 2015

restartしてtravis自体は通りました。
こちらで問題なさそうでしょうか。
よろしくお願いいたします。

@fkanehiro
Copy link
Owner

EPLにLGPLのソースを含めるのってどうなんでしょうか?

@k-okada
Copy link
Contributor

k-okada commented Jun 26, 2015

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

Build Log
last 10 lines

[...truncated 2808 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:1786)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:374)

Test FAILed.

@garaemon
Copy link
Contributor

LGPLは「LGPLのプログラムへの動的リンクであればGPL汚染しない」というライセンスだと解釈すると、hrpsysとの間は動的リンクであればLGPLには汚染しないと思います。

たとえばOpenCV(BSD)はffmpeg(LGPL)を中に持っています。https://github.com/Itseez/opencv/tree/master/3rdparty

コンパイル時は同梱されているバージョン --> 実行時はシステムインストールされているバージョン
というのが起きるとやっかいな気がします。

@k-okada
Copy link
Contributor

k-okada commented Jun 28, 2015

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

Build Log
last 10 lines

[...truncated 2810 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:1786)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:374)

Test FAILed.

fkanehiro added a commit that referenced this pull request Jun 29, 2015
@fkanehiro fkanehiro merged commit a941cda into fkanehiro:master Jun 29, 2015
@snozawa
Copy link
Contributor Author

snozawa commented Jun 29, 2015

すいません、お返事遅くなりました。

#683 (comment)
のコメントを受けて、ソースを含めないようにしました。
最新のもので、-USE_QPOASES=ONをcmakeの引数に追加して実行すると、ソースをとってきます。
結果としては、libqpOASES.soがStabilizer、testZMPDistributorに動的リンクされるようになると思います。

#683 (comment)
は、確かに問題ですね。
やはり3rdpartyのような形をhrpsys-baseの外にもっておくのがよいでしょうか。

@k-okada
Copy link
Contributor

k-okada commented Jun 29, 2015

うーん.Sequenceのなかでsvn coはやり過ぎでは?
hrpsys-base/3rdparty
とかがいいのではないでしょうか.

◉ Kei Okada

2015-06-29 14:58 GMT+09:00 Shunichi Nozawa [email protected]:

すいません、お返事遅くなりました。

#683 (comment)
#683 (comment)
のコメントを受けて、ソースを含めないようにしました。
最新のもので、-USE_QPOASES=ONをcmakeの引数に追加して実行すると、ソースをとってきます。
結果としては、libqpOASES.soがStabilizer、testZMPDistributorに動的リンクされるようになると思います。

#683 (comment)
#683 (comment)
は、確かに問題ですね。
やはり3rdpartyのような形をhrpsys-baseの外にもっておくのがよいでしょうか。


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

@snozawa
Copy link
Contributor Author

snozawa commented Jun 30, 2015

コメントありがとうございます。
3rdparty方向で修正してみようと思います。

@snozawa snozawa deleted the add_qpOases branch June 30, 2015 14:06
snozawa added a commit to snozawa/hrpsys-base that referenced this pull request Jul 29, 2015
@snozawa snozawa mentioned this pull request Jul 29, 2015
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.

4 participants