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

eus_caffe #640

Closed
wants to merge 627 commits into from
Closed

eus_caffe #640

wants to merge 627 commits into from

Conversation

s-noda
Copy link
Contributor

@s-noda s-noda commented Oct 31, 2016

[wip] eus caffe #604

のログを綺麗にしたものです.

@s-noda s-noda mentioned this pull request Oct 31, 2016
@s-noda
Copy link
Contributor Author

s-noda commented Oct 31, 2016

重い sample を git-filter-branch で削除して,catkin build できるようにして
概ねやろうと思ってたことはできたと思います.

しかし,ファイルは消えても commit は消えていないので,コミットログはあんまり綺麗ではないですね.
ファイルが変更されていないコミットをスクリプトで一括削除とかできたらいいのですが.

@s-noda
Copy link
Contributor Author

s-noda commented Oct 31, 2016

しばらくたったら無意味なコミットが消えたような気がします.

コミット数の表示はおかしいようですが,
不要なファイルとコミットが消えて(130ぐらい ahead) ビルドも通りました.

もし,需要があるようならマージして構いません.

@s-noda s-noda changed the title [wip] eus_caffe, log clean up eus_caffe Oct 31, 2016
@snozawa
Copy link
Contributor

snozawa commented Nov 1, 2016

ぱっとみとくに問題なさそうですがいかがでしょう
@k-okada, @YoheiKakiuchi

@k-okada
Copy link
Member

k-okada commented Nov 1, 2016

さすがに250コミットは多いかと.
どれぐらいが空コミットだろう.結構な量だと

https://github.com/jwiegley/git-scripts/blob/master/git-remove-empty-commits
http://stackoverflow.com/questions/5324799/git-remove-commits-with-empty-changeset-using-filter-branch

でできるんじゃないかな.

◉ Kei Okada

2016-11-01 10:15 GMT+09:00 Shunichi Nozawa [email protected]:

ぱっとみとくに問題なさそうですがいかがでしょう
@k-okada https://github.com/k-okada, @YoheiKakiuchi
https://github.com/YoheiKakiuchi


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#640 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeG3NWhrKokopbhtre-NfkS7ELHqlrNks5q5pKugaJpZM4KlZl9
.

@s-noda
Copy link
Contributor Author

s-noda commented Nov 1, 2016

手元で gitk すると 130 コミットぐらいで,

github で見ても
https://github.com/s-noda/jsk_control/commits/catkin_eus_caffe/eus_caffe

消えるべきものは消えているきがするので,コミットの数だけ表示がおかしい気がします.
もうちょっと待ってみたら減るかもしれません.

@k-okada
Copy link
Member

k-okada commented Nov 1, 2016

なるほど、あとはsquashしてまとめてもいいモノ?それておも130コミットは重要なのかな.1つだと少ないんじゃないかという気はするけど.

◉ Kei Okada

2016年11月1日 10:46 Noda, Shintaro [email protected]:

手元で gitk すると 130 コミットぐらいで,

github で見ても
https://github.com/s-noda/jsk_control/commits/catkin_eus_caffe/eus_caffe

消えるべきものは消えているきがするので,コミットの数だけ表示がおかしい気がします.
もうちょっと待ってみたら減るかもしれません.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#640 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeG3HUYWMWHRRjpyPvszY7SkJbUMVcDks5q5pn6gaJpZM4KlZl9
.

@s-noda
Copy link
Contributor Author

s-noda commented Nov 1, 2016

試行錯誤の過程も含めてログはすべて残せばいいと僕は思いますが,重要かといわれるとそうでもないので一つにまとめました.

eus_caffe #641

@s-noda s-noda closed this Nov 1, 2016
@s-noda s-noda deleted the catkin_eus_caffe branch November 18, 2016 02:50
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