Aaron Miller

Results 114 comments of Aaron Miller

We support the latest released version of `ply` on PyPI that I'm aware of, which is `3.11`: https://pypi.org/project/ply/#history We'd potentially be open to also supporting from-source installs of unreleased versions...

I think github must have fixed their markdown renderer so this is no longer necessary - this looks right to me now in both [the symforce docs](https://symforce.org/tutorials/ops_tutorial.html) and [github](https://github.com/symforce-org/symforce/blob/main/notebooks/tutorials/ops_tutorial.ipynb) on...

> I did remember something about isolated builds but could not locate it in the python docs. pip does not respect installed versions of packages in `build-system.requires` for PEP517 packages...

For a constant exponent of 2 known at codegen time, your compiler should optimize the `std::pow` to a multiply, so we don't bother special casing that to generate a multiply...

A more concretely wrong example: ```python In [12]: sm.Max(x, sm.exp(x) - 1, (x + 1)**2 / 2 - sm.S.Half).diff(x).subs(x, 0).evalf() Out[12]: 1.5 ``` https://www.wolframalpha.com/input?i=Plot%5B%7BMax%5Bx%2C+Exp%5Bx%5D+-+1%2C+%28x%2B1%29%5E2+%2F+2+-+1%2F2%5D%2C+D%5BMax%5Bx%2C+Exp%5Bx%5D+-+1%2C+%28x%2B1%29%5E2+%2F+2+-+1%2F2%5D%2C+x%5D%7D%2C+%7Bx%2C+-1%2C+1%7D%5D

> Just encountered this as well. I assume this require fixup in the `ruff` packaging configuration, but it raises an interesting question: is there any mechanism in `rules_python` to allow...

Yeah, I think the guide should be clarified here. I understand from this thread and elsewhere that TestPyPI isn't intended to host things like nightlies or per-commit wheels. The guide...

Reviews in this chain: └**https://github.com/symforce-org/symforce/pull/431 [SymForce] Python 3.13 tests/wheels**

| # | head | base | diff | date | summary | | - | - | - | - | - | - | | 0 | [cae4b864](/symforce-org/symforce/commit/cae4b864f13aeecb5094e5fd23622c2a8d173016)...

As far as which is correct, I think the convention everything else uses is more intuitive, plus it's much less of a breakage to change `Values::LocalCoordinates` than everything else. Above...