Skip to content

混んでいても RDP の解像度を下げない - #96

Merged
5ym merged 1 commit into
mainfrom
fix/no-auto-resolution
Nov 2, 2026
Merged

5ym merged 1 commit into
mainfrom
fix/no-auto-resolution

Conversation

@5ym

@5ym 5ym commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

原因

混んでいるときに RDP の解像度を 75%・50% に下げていた。Windows は倍率 100% のまま画素を減らすので、窓に合わせて引き伸ばすと字やウィンドウが大きく見えていた。

直し方

  • 解像度は窓の大きさのままにする
  • 送る量はサーバーが Frame Acknowledge の queueDepth を見て減らす
  • 混み具合の測定 (FrameLatency) は診断に残す (web/rdp/autoscale.ts → web/rdp/latency.ts)

確かめ

Hyper-V の試験 VM で帯域を 200kbps に絞った。

  • 直す前: 698x243 → 348x200 に下がった
  • 直したあと: 遅延 +1.4 秒でも 698x243 のまま

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU

回線や端末が追いつかないとき、解像度を 100% → 75% → 50% と下げていた。Windows は表示の倍率を
100% のまま画素だけ減らすので、窓いっぱいに引き伸ばすと字やウィンドウがどんどん大きく見えた。
送る量を減らすのはサーバーに任せる(Frame Acknowledge で未描画のフレームの数を返している)。
混み具合の測定は診断に残し、autoscale.ts を latency.ts にした。

Hyper-V の試験用 VM で帯域を 200kbps に絞って確かめた。直す前は 698x243 → 348x200 に下がり、
直したあとは遅延が +1.4 秒溜まっても 698x243 のまま。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU
@claude

claude Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 18s —— View job


レビュー結果

結論: 問題なし

  • autoscale / AutoScale / scaleSwitches / currentScale への参照は残っていません。削除は漏れなく行われています。
  • resize は幅の偶数化と「同じ大きさなら送らない」という既存の条件を保っています。倍率の計算を外しただけなので、挙動の退行は見当たりません。
  • FrameLatency と gfxTimestampMs は autoscale.ts から latency.ts へ移しただけです。gfx.ts の import も更新されています。
  • テストは、残した FrameLatency について、時計のずれ、時計の飛び、日付またぎを引き続き確認しています。縮小ロジックは削除され、それに対応するテストも削除されています。
  • README と docs の記述は、新しい挙動に合っています。
  • セキュリティ上の懸念はありません。

補足(要確認): 検証は Hyper-V の試験 VM で帯域を絞った 1 回です。実環境のサーバーが queueDepth を見て送る量を実際に減らすかは、このリポジトリのコードからは確認できません。

@5ym
5ym merged commit eb5fa1e into main Nov 2, 2026
7 checks passed
@5ym
5ym deleted the fix/no-auto-resolution branch November 2, 2026 16:37
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.

1 participant