From c13a0ab8ebe0c0c5a9d5d6a0e5829206b1534e96 Mon Sep 17 00:00:00 2001 From: josix Date: Mon, 16 Dec 2024 18:34:27 +0800 Subject: [PATCH 1/3] feat: Add aider translation support Co-Authored-By: Wei-Hsiang (Matt) Wang Co-Authored-By: Becca --- .gitignore | 1 + .scripts/intercept.py | 42 ++++++++++++++++++++++++++++++++++++++++++ Makefile | 13 +++++++++++++ 3 files changed, 56 insertions(+) create mode 100644 .scripts/intercept.py diff --git a/.gitignore b/.gitignore index 948c289ec6..d6a797abc0 100644 --- a/.gitignore +++ b/.gitignore @@ -10,3 +10,4 @@ venv/ ENV/ env.bak/ venv.bak/ +.aider* diff --git a/.scripts/intercept.py b/.scripts/intercept.py new file mode 100644 index 0000000000..e4fc437d48 --- /dev/null +++ b/.scripts/intercept.py @@ -0,0 +1,42 @@ +# /// script +# dependencies = [ +# "polib", +# ] +# /// +import argparse +from pathlib import Path + +import polib + + +def get_pofile_from_path(path: Path) -> polib.POFile: + if not path.exists(): + raise ValueError(f"The path '{path.absolute()}' does not exist!") + + if not (path.is_file() and path.suffix == ".po"): + raise ValueError(f"{path} doesn't seem to be a .po file") + + try: + pofile = polib.pofile(path) + except OSError: + raise ValueError(f"{path} doesn't seem to be a .po file") + return pofile + + +if __name__ == '__main__': + parser = argparse.ArgumentParser() + parser.add_argument( + "path", + help="the path of a PO file", + ) + parser.add_argument("-n", '--occurrence_number', type=int, default=1) + args = parser.parse_args() + path = Path(args.path).resolve() + pofile = get_pofile_from_path(path) + occurrence_number = args.occurrence_number + + for entry in pofile: + if not any(path.stem in p and int(n) == occurrence_number for p, n in entry.occurrences): + continue + print(entry.msgid) + break diff --git a/Makefile b/Makefile index f255eb901d..57fbc9dff1 100644 --- a/Makefile +++ b/Makefile @@ -100,6 +100,9 @@ $(VENV)/bin/sphinx-lint: $(VENV)/bin/activate $(VENV)/bin/blurb: $(VENV)/bin/activate . $(VENV)/bin/activate; python3 -m pip install blurb +$(VENV)/bin/aider: $(VENV)/bin/activate + . $(VENV)/bin/activate; python3 -m pip install aider-chat + .PHONY: upgrade_venv upgrade_venv: $(VENV)/bin/activate ## Upgrade the venv that compiles the doc @@ -156,6 +159,16 @@ rm_cpython: ## Remove cloned cpython repo lint: $(VENV)/bin/sphinx-lint ## Run sphinx-lint $(VENV)/bin/sphinx-lint --enable default-role +.PHONY: translate +translate: $(VENV)/bin/aider ## Run translation with aider. Usage: make translate FILE=path/to/file.po LINE=number + @if [ -z "$(FILE)" ] || [ -z "$(LINE)" ]; then \ + echo "\x1B[1;31mError: Both FILE and LINE arguments are required.\x1B[0m"; \ + echo "Usage: make translate FILE=path/to/file.po LINE=number"; \ + exit 1; \ + fi + $(eval MESSAGE=$(shell python3 .scripts/intercept.py $(FILE) -n $(LINE))) + aider --no-auto-commits --message 'Translate the following Python documentation into Tranditional Chinese for $(FILE):$(LINE) with message $(MESSAGE). Ensure that the translation is accurate and uses appropriate technical terminology. The output must be in Traditional Chinese. Pay careful attention to context, idiomatic expressions, and any specialized vocabulary related to Python programming. Maintain the structure and format of the original documentation as much as possible to ensure clarity and usability for readers.' $(FILE) + # This allows us to accept extra arguments (by doing nothing when we get a job that doesn't match, rather than throwing an error) %: @: From 8ad024bca599987a69e99aedcfe76e612bb3ec1b Mon Sep 17 00:00:00 2001 From: Josix Date: Sat, 15 Mar 2025 16:02:09 +0800 Subject: [PATCH 2/3] Update .scripts/intercept.py Co-authored-by: Ezio Melotti --- .scripts/intercept.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/.scripts/intercept.py b/.scripts/intercept.py index e4fc437d48..8878f138f6 100644 --- a/.scripts/intercept.py +++ b/.scripts/intercept.py @@ -25,13 +25,12 @@ def get_pofile_from_path(path: Path) -> polib.POFile: if __name__ == '__main__': parser = argparse.ArgumentParser() - parser.add_argument( - "path", - help="the path of a PO file", - ) - parser.add_argument("-n", '--occurrence_number', type=int, default=1) + parser.add_argument("path", type=Path, + help="the path of a PO file") + parser.add_argument("-n", '--occurrence_number', + type=int, default=1) args = parser.parse_args() - path = Path(args.path).resolve() + path = args.path.resolve() pofile = get_pofile_from_path(path) occurrence_number = args.occurrence_number From faf0311682f1542bf0aa9c426e85f4ac7eaaf089 Mon Sep 17 00:00:00 2001 From: josix Date: Sat, 15 Mar 2025 16:04:50 +0800 Subject: [PATCH 3/3] fixup! feat: Add aider translation support --- .scripts/intercept.py | 25 ++++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/.scripts/intercept.py b/.scripts/intercept.py index 8878f138f6..1a9adb764e 100644 --- a/.scripts/intercept.py +++ b/.scripts/intercept.py @@ -1,8 +1,17 @@ -# /// script -# dependencies = [ -# "polib", -# ] -# /// +""" +A utility script to extract message IDs from PO files. + +This script extracts a specific message ID from a PO file based on the file name +and occurrence number. It's useful for retrieving translation strings for +specific occurrences in the Python documentation. + +Usage: + python intercept.py path/to/file.po [-n OCCURRENCE_NUMBER] + +Arguments: + path: Path to a PO file + -n, --occurrence_number: The occurrence number to match (default: 1) +""" import argparse from pathlib import Path @@ -24,11 +33,13 @@ def get_pofile_from_path(path: Path) -> polib.POFile: if __name__ == '__main__': - parser = argparse.ArgumentParser() + parser = argparse.ArgumentParser( + description="Extract message IDs from PO files" + ) parser.add_argument("path", type=Path, help="the path of a PO file") parser.add_argument("-n", '--occurrence_number', - type=int, default=1) + type=int, default=1, help="the occurrence number to match") args = parser.parse_args() path = args.path.resolve() pofile = get_pofile_from_path(path)