Oscar Benjamin
Oscar Benjamin
> You may now see a "Try it" button under the Examples section under the docstrings for the API elements Okay I see so this is only for autodoc docstrings...
I think this looks nice. I don't know if others have feedback to give...
> Also, there is a maintenance burden (just like sympy live) What do you mean by a maintenance burden? Are you referring to the fact that the old sympy live/gamma...
> I mean that, as far as I know, no one on the SymPy development team knows much about this new setup, so if it breaks someone has to fix...
> Are you suggesting there is no maintenance to this at all? Updating SymPy versions sounds like a maintenance task. It is a maintenance task. It would be an easy...
> * Is this request for documentation in its current state a blocker towards merging this PR? No it is not. Also pretty much all of what I wanted is...
The issue is that given ``` s = -Sum(sin(x/64)*binomial(51, x)*binomial(77, 64 - x)/23951146041928082866135587776380551750, (x, 0, 64)) ``` Then we have: ```python In [31]: s Out[31]: 64 _____ ╲ ╲ ╲...
The merge conflicts need to be resolved.
> The merge conflicts need to be resolved. I have fixed the merge conflicts that were from gh-28177 so now CI checks will run in the PR. There are test...
```diff diff --git a/sympy/polys/rings.py b/sympy/polys/rings.py index 02ad07f171..8030ebd0ac 100644 --- a/sympy/polys/rings.py +++ b/sympy/polys/rings.py @@ -4041,8 +4041,11 @@ def clear_denoms(self): def _change_ring(self, new_ring): a = self.to_PythonPolyElement() - result = a._change_ring(new_ring) - return...