Fix and test for misinterpreted dimensionless power of exponent
PR #24066 ↗ · sympy/sympy · · merged Oct 24, 2022 · +24−4 · base 514579c655bf
Agent prompt
what a new run launched now would send
SI._collect_factor_and_dimension() cannot properly detect that exponent is dimensionless
How to reproduce:
```python
from sympy import exp
from sympy.physics import units
from sympy.physics.units.systems.si import SI
expr = units.second / (units.ohm * units.farad)
dim = SI._collect_factor_and_dimension(expr)[1]
assert SI.get_dimension_system().is_dimensionless(dim)
buggy_expr = 100 + exp(expr)
SI._collect_factor_and_dimension(buggy_expr)
# results in ValueError: Dimension of "exp(second/(farad*ohm))" is Dimension(time/(capacitance*impedance)), but it should be Dimension(1)
```
Work only inside this repository checkout. Make the code change the task
describes, keeping the diff focused — no drive-by refactors.
When you are done, leave your changes committed or in the working tree;
they are collected automatically.
Stay on this snapshot checkout (`task/ycb_sympy_pr24066`). Never checkout, pull, or rebase onto `main`. That branch is a README-only orphan.
Stay on this HEAD. Do not fetch another default branch. Push only on the Cursor-created `crazy-cursor/…` side branch from this HEAD.
Some past runs of this task were launched with a different prompt (the prompt template changed since, or those runs predate this benchmark's stored prompt). Each run persists the exact prompt it sent at launch — that per-launch record is the audit trail; this page shows only the current one.
Reference diff
The task PR's merged diff — the judge's comparison target.