mirror of
https://github.com/NawfalMotii79/PLFM_RADAR.git
synced 2026-05-14 11:51:47 +00:00
49055a8bf1
Two independent root causes landed together in the recent develop merge
burst, leaving CI red on every commit since. Both fixes are narrow parser/
config updates; no code logic changes.
1. Ruff T201 violation in 8_Utils/Python/LUT.py:24
The recent restoration of `print(f"8'd{val},")` (replacing `pass`) made
the LUT generator functional again, but the file is not in pyproject's
per-file-ignores so ruff's T20 rule flags it. Add a narrow per-file-
ignore for T20 specifically scoped to LUT.py.
2. Five tests in TestTier1Adar1000ChannelRegisterRoundTrip
The recent SPI/ADC error-propagation refactor on ADAR1000_Manager.cpp
changed the four setters adarSet{Rx,Tx}{Phase,VgaGain} from `void` to
`bool` returns AND introduced the `ok = setter(args) && ok` error-chain
idiom at internal call sites. The test class's parser regexes were
authored for the pre-refactor convention.
Two regex extensions:
* Body parser: `void\s+...` -> `(?:void|bool)\s+...` so bodies are
found under either return-type convention.
* Caller finder: `\)\s*;` -> `\)(?:\s*&&\s*\w+)*\s*;` so the
error-chain idiom is matched alongside standalone calls.
Body extraction logic, REG_CHn_XXX + <expr> regex, symbolic AST
evaluation, round-trip strict-equality, and stride detection are all
unchanged. The two regex extensions only acknowledge the new C++
conventions introduced by the SPI/ADC refactor.
Verified locally:
- `uv run ruff check .` returns 0 errors.
- `uv run pytest 9_Firmware/tests/cross_layer/test_cross_layer_contract.py`
passes 51 tests + 5 skipped (Tier2VerilogCosim local-only, requires
iverilog which is installed in CI).
- `uv run pytest 9_Firmware/9_3_GUI/test_v7.py` passes 83 tests + 3
skipped (no regression).
55 lines
2.4 KiB
TOML
55 lines
2.4 KiB
TOML
[project]
|
|
name = "aeris-10-radar"
|
|
version = "1.0.0"
|
|
description = "AERIS-10 FMCW Radar Platform — host software & FPGA cosim tools"
|
|
requires-python = ">=3.12"
|
|
|
|
# Runtime dependencies intentionally empty — GUI deps are optional and
|
|
# listed in requirements_*.txt files for local installs.
|
|
dependencies = []
|
|
|
|
[dependency-groups]
|
|
dev = [
|
|
"ruff>=0.5",
|
|
"pytest>=8",
|
|
"numpy>=1.26",
|
|
"h5py>=3.10",
|
|
]
|
|
|
|
# ---------------------------------------------------------------------------
|
|
# Ruff configuration
|
|
# ---------------------------------------------------------------------------
|
|
[tool.ruff]
|
|
target-version = "py312"
|
|
line-length = 100
|
|
|
|
[tool.ruff.lint]
|
|
select = [
|
|
"E", # pycodestyle errors
|
|
"F", # pyflakes (unused imports, undefined names, duplicate keys, assert-tuple)
|
|
"B", # flake8-bugbear (mutable defaults, unreachable code, raise-without-from)
|
|
"RUF", # ruff-specific (unused noqa, ambiguous chars, implicit Optional)
|
|
"SIM", # flake8-simplify (dead branches, collapsible ifs, unnecessary pass)
|
|
"PIE", # flake8-pie (no-op expressions, unnecessary spread)
|
|
"T20", # flake8-print (stray print() calls — LLMs leave debug prints)
|
|
"ARG", # flake8-unused-arguments (LLMs generate params they never use)
|
|
"ERA", # eradicate (commented-out code — LLMs leave "alternatives" as comments)
|
|
"A", # flake8-builtins (LLMs shadow id, type, list, dict, input, map)
|
|
"BLE", # flake8-blind-except (bare except / overly broad except)
|
|
"RET", # flake8-return (unreachable code after return, unnecessary else-after-return)
|
|
"ISC", # flake8-implicit-str-concat (missing comma in list of strings)
|
|
"TCH", # flake8-type-checking (imports only used in type hints — move behind TYPE_CHECKING)
|
|
"UP", # pyupgrade (outdated syntax for target Python version)
|
|
"C4", # flake8-comprehensions (unnecessary list/dict calls around generators)
|
|
"PERF", # perflint (performance anti-patterns: unnecessary list() in for loops, etc.)
|
|
]
|
|
|
|
[tool.ruff.lint.per-file-ignores]
|
|
# Tests: allow unused args (fixtures), prints (debugging), commented code (examples)
|
|
"test_*.py" = ["ARG", "T20", "ERA"]
|
|
# Re-export modules: unused imports are intentional
|
|
"v7/hardware.py" = ["F401"]
|
|
# 8_Utils/Python/LUT.py is a single-purpose Verilog LUT generator whose only
|
|
# output is `print(f"8'd{val},")` per the file's docstring intent.
|
|
"8_Utils/Python/LUT.py" = ["T20"]
|