-
Notifications
You must be signed in to change notification settings - Fork 215
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
EPUBビルダの処理時間が多くなった(V3.2.0→V5.9.0 約6倍) #1915
Comments
画像ファイルの処理に時間を要している?config.ymlの
preface.re直後の44秒が1秒となりました。 |
WSL2 Windowsファイルアクセスに時間を要している?WSL2からWindowsファイルにアクセスすると遅いらしいです。
Ver3.2.0の時は、Windowsファイルにアクセスしていも速いけど、Ver5.9.0では6倍必要。
|
過去にimage finderが遅い #1896 ということでチューニングはしたものの、類似の現象が発生しているのかな…と推測はしました。
|
サンプルでの計測現象が発生している原稿から公開できない部分を削除しました。
画像ファイルは、192個あります。 本文がないことと画像ファイルが小さくなったことで幾分速くなってしまいましたが、V3.2.0に比べて2倍の処理時間を要しています。
サンプルの提供 |
Re:VIEW 5からIndexBuilderを導入しています https://review-knowledge-ja.readthedocs.io/ja/latest/releases/review500.html#58f2b4008c87db7e7d85a9f5e993c209 。 ただ、いただいた情報で拝見すると、そもそもの3.2の時間も妙にかかっていますし、real-(user+sys)が22.161秒ということはCPUを使わない処理、おそらくI/O待ちが予想され、WSLのI/Oパフォーマンスに問題があるのではないかと予想します。 https://learn.microsoft.com/ja-jp/windows/wsl/compare-versions によると、WSL/WSL2はマウントされたファイル(おそらく/mnt)についてI/Oパフォーマンスの低下が生じる仕様に見えます。試しにサンプルフォルダの内容を/mntでない別の場所(Windowsマウントでない場所)にコピーし、そこで実行してみることで、WSL I/O起因かどうかを判断できるかと思います。 WSL I/O起因だった場合はRe:VIEW側では手立てはなく、Dockerを試してみていただくなどの代替手段となりそうです。 |
お手数おかけして申し訳ありませんでした。 IndexBuilder導入により先に参照を構成して、都度参照しないようにした結果、6倍遅くなってしまったことは残念です。 どうもありがとうございました。 |
あくまでも推測なので、
もしてみていただければと思います。 |
アドバイス、ありがとうございました。 提供したサンプルプロジェクトを/mnt以外に配置すると 0m1.315sでEPUBビルドできました。 なんとなく、WSL2の問題なのかと推測していましたが、Ver3.2.0に比べて遅くなってしまった原因が不明でした。 EPUBビルドをLinux側で行う方針に変更します。 このたびは、ありがとうございました。 |
はい、ご報告ありがとうございました! |
EPUBビルダの処理時間
Ver3.2.0で作成した文章をVer5.9.0でEPUBビルドすると、処理時間が多くなりました。
同じPCですが、環境が異なります。
環境の問題ならあきらめますが、もう少し速くならないものでしょうか。
DEBUGログ
DEBUGログを見ていると、最初の.re(preface.re)を処理した後に44秒かかっています。
preface.reをcatalog.ymlから外すとchapter_01.reの後で処理時間がかかっています。
すなわち、最初の.reを処理した後に処理時間がかかっているように見えます。
ここでは、どんな処理を行っているのでしょうか。
DEBUGログの改造
DEBUGログに時間を追加するために下記のように改造しています。
/var/lib/gems/3.2.0/gems/review-5.9.0/lib/review/loggable.rb
The text was updated successfully, but these errors were encountered: