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

進捗表示を単純化してほしい #3

Closed
ginpei opened this issue Feb 24, 2016 · 8 comments
Closed

進捗表示を単純化してほしい #3

ginpei opened this issue Feb 24, 2016 · 8 comments

Comments

@ginpei
Copy link

ginpei commented Feb 24, 2016

v5から? npm install 時に進捗を示すバーが表示されるようになったと思いますが、これをもっと単純なものに置き換えてほしいです。

私の環境だと文字幅の解釈がおかしいのか、このように表示されてしまいます。

cwnib-pueaemlcq

他にもtmuxで画面分割した際、右隣の画面へはみ出すという話も聞いた事があります。

本質的にnpmの不具合というわけではないとは思いますが、例えば wget の表示のように単純なものになるとか、いっそ過去のバージョンのように棒が回転するやつ(進捗表示ではありませんが)に戻ると嬉しく思います。

直近の回避策

端末の文字表示関連の設定でも回避できるかと思いますが、手元では npm set progress=false の設定して進捗を表示しないようにしました。本現象に困っており、かつ進捗表示を必要としない方はお試しください。

ちなみに非表示にすると高速化されるという話もあります。(ただし私の環境では特に差異はなさそうでした。)

@mysticatea
Copy link

これ、現行の npm (v3.7.x) だと昔ながらの ASCII 表現になってて、良い感じでした。

@watilde
Copy link
Owner

watilde commented Mar 5, 2016

こちらの問題、倍角文字・東アジアの文字の幅の扱いに起因していることが多い印象です。

私の環境だと文字幅の解釈がおかしいのか、このように表示されてしまいます。

RLoginだと、一行に表示したい文字数に合わせてフォントサイズが変わるという設定があるようですね。設定画面で 「スクリーン」 > 「フォントサイズから一行あたりの文字数を決定」に変更したら直ったりしませんでしょうか?


速度に関して。progress barの表示に関してbugがあったため遅かったんですが、 v3.7.0 より高速化されました。とはいえ、表示しない方が早いのは確かなので、もし不要であればオフにする運用でも良いと思います;

関連リンク:

@watilde
Copy link
Owner

watilde commented Mar 5, 2016

@ginpei 追記です。npm側での改善の余地として、unicode optionがprogress barに適用されていなかったバグを見つけました。該当箇所を修正し、issueの報告とpull requestの作成をしてみました。こちらを以て様子見とさせてください 😃

@ginpei
Copy link
Author

ginpei commented Mar 6, 2016

ありがとうございますー。

すみませんパッケージの更新は忘れていました……。結果的に、v3.7.5, v3.8.0共に状況に変化はないようです。
v3.7.5

@mysticatea
Copy link

これ v3.7.5 です。
なるほど、Windows だとデフォルトで unicode: false なんですね。

npm

@watilde
Copy link
Owner

watilde commented Mar 7, 2016

下記の処理でfallbackに倒れるOSはシンプルなプログレスバーになりますね。
https://github.com/iarna/gauge/blob/master/themes.js#L10-L12

ちなみに今回見つかった問題は、
npm config set unicode false していてもasciiのプログレスバーに切り替わらないケースがあるというものでした。報告いただいたissueはOSに関わらず起きうる問題なので、optionで避けられるようにしたいところ。。

@watilde
Copy link
Owner

watilde commented Mar 19, 2016

@ginpei v3.8.2 がstableとしてリリースされました。下記のコマンドを実行することで、進捗表示を単純化が可能になりました 😃

$ npm install -g npm
$ npm config set unicode false

@watilde watilde closed this as completed Mar 19, 2016
@ginpei
Copy link
Author

ginpei commented Mar 19, 2016

良さそうです。ありがとうございました!

lgtm

watilde pushed a commit that referenced this issue Feb 7, 2020
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

3 participants