Fix uncaught ValueError in flatten_pass_by_value on malformed hex input

[Anerudhan]

The hex branch of flatten_pass_by_value converted "0x"-prefixed strings without error handling, so malformed values such as "0x" or "0xZZ" in a log's pass_by_value field crashed the cudnn_repro CLI with an unhandled ValueError. Guard the conversion with the same try/except pattern the decimal branch already uses, returning an empty list for unparseable strings, and add regression tests.

Fixes #342

Summary by CodeRabbit

• Bug Fixes

  • Improved parsing of numeric strings so values with a 0x prefix are handled as hexadecimal, while other strings continue to be treated as decimal.
  • Non-convertible values still safely return an empty result, preserving existing behavior.

• Tests

  • Added coverage for valid hex and decimal inputs, malformed strings, mixed lists, and None/numeric values to verify parsing behavior across common cases.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: b2a961a5-17b7-4214-a710-25597d59b049

📥 Commits

Reviewing files that changed from the base of the PR and between 52119ee and dea890c.

📒 Files selected for processing (2)

• tools/cudnn_repro/cudnn_repro/utils.py
• tools/cudnn_repro/tests/test_cudnn_repro_utils.py

---

📝 Walkthrough

Walkthrough

The flatten_pass_by_value function in cudnn_repro/utils.py was refactored to determine an integer base from the string prefix (0x → 16, else → 10) and perform conversion in a single guarded try/except ValueError block. A new test file was added covering valid and malformed inputs.

Changes

flatten_pass_by_value fix and tests

Layer / File(s)	Summary
Guarded hex/decimal parsing tools/cudnn_repro/cudnn_repro/utils.py	Unified hex and decimal string parsing into one try/except ValueError block, fixing an uncaught crash on malformed 0x-prefixed strings.
Unit tests for parsing behavior tools/cudnn_repro/tests/test_cudnn_repro_utils.py	Adds seven tests covering valid hex/decimal, malformed hex/decimal, mixed lists, and None/numeric inputs.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Related issues

• #342: Fixes uncaught ValueError in flatten_pass_by_value on malformed hex pass_by_value strings.

Suggested labels: bug, tools

Suggested reviewers: (none identified)

🐰 A hex string once made me crash,
Now caught in a try/except stash,
Base sixteen or ten,
Parsed safely again,
Tests confirm—no more traceback flash!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the malformed-hex ValueError fix in flatten_pass_by_value.
Linked Issues check	✅ Passed	The code now handles malformed hex like decimal parsing and adds regression tests for 0x and 0xZZ.
Out of Scope Changes check	✅ Passed	The changes stay focused on the crash fix and regression coverage; no unrelated behavior was introduced.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}