Introduce `json_loads` executions benchmark by moreal · Pull Request #6723 · RustPython/RustPython

Walkthrough

A new JSON loading benchmark script is added that reads and parses a JSON file from the data directory using the standard library's json module to measure JSON deserialization performance.

Changes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A JSON file hops in with glee,
Loads parse swift as can be,
With benchmark so keen,
Performance we'll glean,
Speedier parsing for you and me! 📊

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

• 📝 Generate docstrings

• X
• Mastodon
• Reddit
• LinkedIn

benches/benchmarks/json_loads.py (2)

3-4: Consider specifying explicit UTF-8 encoding.

JSON files are UTF-8 by specification. Without an explicit encoding parameter, Python uses the system's default encoding, which may differ on some platforms.

Suggested improvement

-with open('benches/_data/pypi_org__simple__psutil.json') as f:
+with open('benches/_data/pypi_org__simple__psutil.json', encoding='utf-8') as f:
     data = f.read()

---

5-7: Remove extra blank line.

PEP 8 recommends a single blank line between logical sections within a module. There are two consecutive blank lines here.

Suggested fix

 with open('benches/_data/pypi_org__simple__psutil.json') as f:
     data = f.read()

-
 loaded = json.loads(data)

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro