LiveSplit / LiveSplit/SpeedrunComSharp
Optimize Embeds away if the non-embeds are already queried
まだ誰も着手していません。
- 主要言語
- C#
- スター
- 32
- フォーク
- 11
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Embeds seem to be useful as they reduce the amount of API calls needed, but at the same time, they can cause issues with the current Caching implementation, as the elements might already be queried, but the embedded API call has not been done yet. So it causes more queries instead of less queries.
Simply stripping the ?embeds=... from the URL when comparing, doesn't work, because the embedded objects might not be queried yet. If there's more than 1 embedded object that it wants to access, it would cause more queries then. So this isn't really a good solution.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、現在の Caching 実装と、?embeds=... を含むリクエスト URL がどのように比較されるかを追跡します。すでにクエリ済みの非埋め込みオブジェクトと複数の埋め込みオブジェクトに対する動作を定義し、そのうえで、必要な埋め込みデータをスキップせずに冗 redundant な API 呼び出しが回避されることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- csharp
- 領域
- api, backend
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100