Remote dataset loader leaks a temp file per cache=False fetch
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
_fetch_from_url から始めて、その一時ファイルの処理を #1894 のメンテナーによる修正と比較してください。cache=False の fetch パスが、エラー後も含め、完了時に一時的なデータセットファイルをクリーンアップすることを確認し、周辺のテストに適切な場所がある場合は、キャッシュを使用しない fetch の繰り返しに対するカバレッジを追加または更新してください。
索引モデルが issue の本文から書いたものです。
説明
what happens
the remote dataset loader leaks a full temp file on every cache=False fetch. _fetch_from_url writes the downloaded dataset into a NamedTemporaryFile(delete=False) and then abandons the handle without closing or deleting it, so each non-cached fetch strands a complete copy of the dataset in the temp directory.
this is the same bug class the maintainers already fixed for save_formatted_audio in #1894.
expected
the temp file is cleaned up when the fetch path is done with it (context-managed handle, or explicit close plus unlink in a finally), matching the #1894 fix pattern.
env: main 5503ecb
- 主要言語
- Python
- スター
- 4.5k
- フォーク
- 896
- 平均マージ
- 3日 1時間
- マージ済み PR（30日）
- 208
環境構築
このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/PyRIT のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
microsoft/PyRIT の issue をすべて見る
似ている issue
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
kristofdegrave/homeassistant-smart-charging#1413 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
nasa/earthdata-varinfo#113 ·
-
curriculum documentation quality
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
githubnext/gh-aw-workshop#3849 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信