機能 #1038: TLに表示するツイート上限数をユーザー設定で変更可能にしたい - mikutter - やること

操作

リンクをコピー

機能 #1038

完了

TLに表示するツイート上限数をユーザー設定で変更可能にしたい

Izumi Tsutsui さんが7年以上前に追加. 約7年前に更新.

ステータス:

終了

優先度:

通常

担当者:

対象バージョン:

3.6

開始日:

2017-05-06

期日:

進捗率:

プラグイン名:

説明

現状の表示ツイート数上限は magic number で埋め込んである以下の数字だと思います。

source:core/mui/cairo_timeline.rb@27049201#L168

  # TLのMessageの数が上限を超えたときに削除するためのキューの初期化
  # オーバーしてもすぐには削除せず、1秒間更新がなければ削除するようになっている。
  def init_remover
    @timeline_max = 200
    @remover_queue = TimeLimitedQueue.new(1024, 1){ |messages|
      Delayer.new{
        if not destroyed?
          remove_count = size - timeline_max
          if remove_count > 0
            to_enum(:each_iter).to_a[-remove_count, remove_count].each{ |iter|
              tl_model_remove(iter) } end end } } end

とりあえず感がありますが、上記の 200 の数字を userconfig で変更可能なようにするという案はどうでしょう?

200という数字は mikutter 黎明期になんとなくで決められた値のように見えます。
https://twitter.com/toshi_a/status/9691075197

リビジョン12をコミットしました：
タイムライン上のつぶやきの完璧なソート
長すぎるつぶやきの末尾を切って投稿
メモリ節約のため、２００個以上前のつぶやきを非表示に
READMEの後進を忘れなかった
明日起きれるか心配になった
#mikutter
3:48 - 2010年2月27日

どんなもんでしょう。

ファイル

一括ダウンロード

mikutter-timeline_max-settings.png (77 KB) mikutter-timeline_max-settings.png		Izumi Tsutsui, 2017-11-12 18:05
mikutter-timeline_tweet-number.png (64.8 KB) mikutter-timeline_tweet-number.png		Izumi Tsutsui, 2017-11-12 18:06

関連するチケット

操作

リンクをコピー

toshi_a 初音さんが7年以上前に更新

対象バージョン を 3.5 から 3.6 に変更

賛成です。
200という数字の根拠は、当時Twitter APIで各種タイムラインを取得する時に、1ページの最大件数が200だったことに由来していたと記憶しています。
cairoを使った現在の実装だと、一度表示されたことのある画像がメモリ上に残るが、処理速度的にはペナルティは少ないと思うので、よりスペックの高い環境であればこれを増やすことが出来ると便利そうです。

3.5からは、filerプラグインのように200以上のModelを常に保持するのが正当なケースも出てきたので、UIDSLのtimelineメソッドで、プラグインがタイムラインを生成する時に最大件数を設定できるようにしようと思います。これのデフォルト値は、UserConfigに記録しておいて、エンドユーザが設定できるようにすると良いでしょう。

操作

リンクをコピー