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

use a different color for @knsmr from @kdmsnr #51

Closed
wants to merge 1 commit into from

Conversation

irohiroki
Copy link

デフォルトの状態だと西村さんも角さんも青で、見分けが付きません。
このパッチはどんな色設定でも必ず西村さんと角さんを別の色で表示します。
また、西村さん以外の人はパッチを当てる前の色のままです。

(冗談なのでmergeしないでくださいw)

@irohiroki
Copy link
Author

あ… 「どんな色設定でも」はウソだ…
config[:colors]に1色しか入ってなかったらダメだw

@jugyo
Copy link
Owner

jugyo commented Jul 1, 2011

悩ましい問題ですねぇ。
これは明らかに earthquake.gem のバグですが、
修正するとしても影響範囲がでかすぎるので( @knsmr さんと @kdmsnr さんを同一人物だと思っている人がいる可能性もあって、いきなり色が変わるとそういったユーザーを混乱させてしまうかもしれません)、
とりあえず @knsmr さん @kdmsnr さんのどちらかにアカウント名を変えてもらうようお願いしてみましょう。

@ujihisa
Copy link

ujihisa commented Jul 1, 2011

ハミング距離の近い任意の2つの文字列が同じハッシュ値をもたないようにするアルゴリズムに関する論文が民明書房で紹介されていたので、参考になるかもしれません。

@knsmr
Copy link

knsmr commented Jul 1, 2011

私を特別扱いするパッチはいただけませんね、プログラマなら問題を汎化して解決しないと ;)
というわけで、ハミング距離でも編集距離でもいいですが、「似た名前」が必ず違う色になるようなパッチ希望です。
実は人間は単語を語頭の1文字と語尾の1文字で認識しているので、そのロジックも必要ですよね。

http://d.hatena.ne.jp/yarb/20081116/p1

つまり「見間違えやすい名前」をスコアリングしないとですが、よく考えると、その人のタイムラインに登場する頻度も考慮すべきですよね。

@jugyo
Copy link
Owner

jugyo commented Jul 1, 2011

設定ファイル等でユーザーの色を明示的に指定できるようにするというのも現実的な解決策かもしれません。

Earthquake.config[:users]['knmsr'][:color] = 'red'

みたいな感じで

@ujihisa
Copy link

ujihisa commented Jul 2, 2011

名前の色付けロジックはearthquakeに限らず汎用的に使いたいので、独立したgemで存在するとすごく便利ですね。

@no6v no6v closed this in 2409f2d Jan 21, 2012
@no6v
Copy link
Collaborator

no6v commented Jan 21, 2012

文字列から英数字を抜き出して 36 進数として数値化してるだけなので、
実は最後の一文字だけで色が決まっちゃうんです。
なので、個別に色を指定出来るようにしてみました。
続きは #102 の方でお願いします。

@jugyo
Copy link
Owner

jugyo commented Jan 21, 2012

そういうことだったんですね。
デフォルトの動きとして、最後の一文字よりかは最初の一文字で色が決まったほうがいい気もしてきました。

@no6v
Copy link
Collaborator

no6v commented Jan 21, 2012

first をはさんでみたら、ちょっと世界が変わりましたw
けど、それほどギャッって感じでもないですね。

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.

None yet

5 participants