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

プログレスバーが動作するたびに改行されていく #7

Closed
kmuto opened this issue Oct 17, 2023 · 8 comments
Closed

プログレスバーが動作するたびに改行されていく #7

kmuto opened this issue Oct 17, 2023 · 8 comments

Comments

@kmuto
Copy link

kmuto commented Oct 17, 2023

仕様なのか判断がつかなかったのですが、プログレスバーが進捗するたびに改行されていくようです(macOS)。

@tukaelu
Copy link
Owner

tukaelu commented Oct 17, 2023

自分もmac環境で再現しないので恐らくターミナル環境の問題かと思われます。(仕様ということはないです)
schollz/progressbarを使用しているのですが、関連しそうなIssueがあるかはちょっと確認してみます。

あと念のため以下のような情報も貰えると嬉しいです。

  • macOSのバージョン
  • ターミナルソフトウェアの種類・バージョン
  • 使用しているシェル環境

@kmuto
Copy link
Author

kmuto commented Oct 20, 2023

  • Ventura 13.6
  • iTerm2 3.4.21
  • /bin/zsh
    となります。

tukaelu added a commit that referenced this issue Oct 22, 2023
I explicitly specified the options to minimize the issue of the progress bar breaking into new lines to the greatest extent possible.

refs #7
@tukaelu
Copy link
Owner

tukaelu commented Oct 22, 2023

@kmuto
自分が使用しているAlacrittyでは再現しなかったのですが、iTerm2やmacOS標準のターミナルで再現することを確認しました。
schollz/progressbar#97 に関連していそうで、まだ未解決みたいですね。

パッチを送るより表示を調整するほうが低コストで最大限回避できそうだったので v0.1.7 で対応してみましたが、ターミナルの表示サイズに依存しそうなので最大限回避としています。ついでに表示内容もちょっとだけわかりやすく調整しております。

スクリーンショット 2023-10-22 16 04 21

@kmuto
Copy link
Author

kmuto commented Oct 22, 2023

kawaii!

iTerm2, macターミナルだと改行されるのはしょうがなさそうですね。


🐟 Donwload host metrics (create 5 file(s))   6% [>                         ]                                                                               

🐟 Donwload host metrics (create 5 file(s))  13% [==>                       ]                                                                               

🐟 Donwload host metrics (create 5 file(s))  20% [====>                     ]                                                                               

🐟 Donwload host metrics (create 5 file(s))  26% [=====>                    ]                                                                               

@tukaelu
Copy link
Owner

tukaelu commented Oct 23, 2023

@kmuto
参考までにターミナルウィンドウごとスクショいただけると 🙏
こちらは再現しないので解像度とか文字サイズの違いとかなのかなぁ。。

@kmuto
Copy link
Author

kmuto commented Oct 23, 2023

キャプチャしてみました
スクリーンショット 2023-10-23 20 27 29

外付けモニタでretinaになってないですね。iTerm2 80x25文字。

@tukaelu
Copy link
Owner

tukaelu commented Oct 23, 2023

@kmuto
なるほど、横幅80(多分デフォルトかも?)だとちょっと狭いですね。。
使用しているライブラリにいくつか問題があり、次のような場合に上記が再現するようでした。

  1. そもそもプログレスバーの行をクリアする処理が誤っていそう
  2. リサイズしても発生する
  3. 表示内容が折り返すくらい幅が狭いと発生する

上記の一部を改善するPRを本家に出しつつ、一旦幅が狭い場合でも(80くらいであれば)上記が発生しないように v0.1.8 で対応したつもりなのでご確認ください。

@kmuto
Copy link
Author

kmuto commented Oct 24, 2023

80はたいていのターミナルでデフォルトですね…
ということで0.1.8で試したところ、横80文字でもあふれずに実行できています!
このissueはcloseいただいて構いません!

@tukaelu tukaelu closed this as completed Oct 24, 2023
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

No branches or pull requests

2 participants