Skip to content

Conversation

qwerty2501
Copy link
Contributor

内容

削減できる量は少ないですがわりと簡単に設定できそうだったのでやってみました

その他

before

> ls -lh libcore.so
-rwxrwxr-x 2 qwerty2501 qwerty2501 62M  8月  7 18:17 libcore.so

after

> ls -lh libcore.so
-rwxrwxr-x 2 qwerty2501 qwerty2501 57M  8月  7 18:26 libcore.so

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

onnxファイルが巨大なので全体的にはあまり変わりませんが、コード部分だけで考えると相当な割合減ってそうですね。

Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

# min-sized-rustを元にrelease buildのサイズが小さくなるようにした
# https://github.com/johnthagen/min-sized-rust
[profile.release]
opt-level = "z"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

横から失礼します。
サイズ優先最適化だとスピードが少し落ちそう…?(落ちたとしても誤差レベルかも知れない)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

たしかに。まあ誤差レベルだと思うので、なんか遅いなってなったら検討しますか。

Copy link
Contributor Author

@qwerty2501 qwerty2501 Aug 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

大部分はonnxruntimeの処理がボトルネックになってるかなと思って設定しました
仮に速度優先だと 3 を設定するようにしたほうが良いです

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