From 85743b4eaf1bd902b6d11b0e5a6a1ede400715e8 Mon Sep 17 00:00:00 2001 From: Aaron Gokaslan Date: Wed, 25 Dec 2024 15:05:57 -0500 Subject: [PATCH 1/2] Use operator.itemgetter for perf/readability in archive --- fsspec/archive.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fsspec/archive.py b/fsspec/archive.py index f466780fc..14f62b5a0 100644 --- a/fsspec/archive.py +++ b/fsspec/archive.py @@ -1,5 +1,6 @@ from fsspec import AbstractFileSystem from fsspec.utils import tokenize +import operator class AbstractArchiveFileSystem(AbstractFileSystem): @@ -67,7 +68,7 @@ def ls(self, path, detail=True, **kwargs): out = {"name": ppath, "size": 0, "type": "directory"} paths[ppath] = out if detail: - out = sorted(paths.values(), key=lambda _: _["name"]) + out = sorted(paths.values(), key=operator.itemgetter("name")) return out else: return sorted(paths) From 3c4f527b9c77f9b16bbcaf6cb880cb05fd3a9278 Mon Sep 17 00:00:00 2001 From: Martin Durant Date: Mon, 6 Jan 2025 11:42:12 -0500 Subject: [PATCH 2/2] lint --- fsspec/archive.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fsspec/archive.py b/fsspec/archive.py index 14f62b5a0..13a4da8df 100644 --- a/fsspec/archive.py +++ b/fsspec/archive.py @@ -1,6 +1,7 @@ +import operator + from fsspec import AbstractFileSystem from fsspec.utils import tokenize -import operator class AbstractArchiveFileSystem(AbstractFileSystem):