Remote dataset loader leaks a temp file per cache=False fetch
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
从 _fetch_from_url 开始,将其临时文件处理方式与 #1894 中维护者的修复进行比较。确认 cache=False fetch 路径在完成后会清理临时 dataset 文件,包括发生错误之后;如果周围的测试提供了合适的位置,则为重复的非缓存 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 小时
- 30 天内合并 PR
- 208
环境准备
我们还没有检查这个项目的环境配置文件。先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 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 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
solana-foundation/pay-kit#341 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
nasa/python_cmr#123 ·
-
难度 1/5 1 小时以内 新手友好度 92/100
EleutherAI/lm-evaluation-harness#4243 ·
维护者通常 1 天内回复
-
area: dashboard bug perceived difficulty: 3
难度 2/5 1-3 小时 新手友好度 88/100
Nitjsefnie-Harness-Commons/daedalus#1179 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 88/100
cusp-ai-oss/tojax#17 ·