Rename autodiff L_op and R_op methods to pull_back and push_forward

[ricardoV94]

Closes #182

L_op -> pull_back
R_op -> push_forward
push_forward: takes outputs as well (consistent signature with L_op)
push_forward: Uses DisconnectType instead of None (used to be the same in L_op, but they changed after a while)

Claude generated

[ricardoV94]

Did you ever see this one @jessegrabowski

FAILED tests/tensor/test_slinalg.py::TestSchur::test_schur_empty - ValueError: negative dimensions not allowed
Apply node that caused the error: Schur{output='real', overwrite_a=False, sort=None}(<Matrix(float64, shape=(?, ?))>)
Toposort index: 0
Inputs types: [TensorType(float64, shape=(None, None))]
Inputs shapes: [(0, 0)]
Inputs strides: [(0, 0)]
Inputs values: [array([], shape=(0, 0), dtype=float64)]
Outputs clients: [[output[0](Schur{output='real', overwrite_a=False, sort=None}.0)], [output[1](Schur{output='real', overwrite_a=False, sort=None}.1)]]

Backtrace when the node is created (use PyTensor flag traceback__limit=N to make it longer):
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_callers.py", line 121, in _multicall
    res = hook_impl.function(*args)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/_pytest/runner.py", line 179, in pytest_runtest_call
    item.runtest()
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/_pytest/python.py", line 1720, in runtest
    self.ihook.pytest_pyfunc_call(pyfuncitem=self)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_hooks.py", line 512, in __call__
    return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_manager.py", line 120, in _hookexec
    return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_callers.py", line 121, in _multicall
    res = hook_impl.function(*args)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/_pytest/python.py", line 166, in pytest_pyfunc_call
    result = testfunction(**testargs)
  File "/home/runner/work/pytensor/pytensor/tests/tensor/test_slinalg.py", line 1363, in test_schur_empty
    T, Z = schur(A)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_callers.py", line 121, in _multicall
    res = hook_impl.function(*args)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/_pytest/runner.py", line 179, in pytest_runtest_call
    item.runtest()
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/_pytest/python.py", line 1720, in runtest
    self.ihook.pytest_pyfunc_call(pyfuncitem=self)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_hooks.py", line 512, in __call__
    return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_manager.py", line 120, in _hookexec
    return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/pluggy/_callers.py", line 121, in _multicall
    res = hook_impl.function(*args)
  File "/home/runner/micromamba/envs/pytensor-test/lib/python3.14/site-packages/_pytest/python.py", line 166, in pytest_pyfunc_call
    result = testfunction(**testargs)
  File "/home/runner/work/pytensor/pytensor/tests/tensor/test_slinalg.py", line 1363, in test_schur_empty
    T, Z = schur(A)

HINT: Use the PyTensor flag `exception_verbosity=high` for a debug print-out and storage map footprint of this Apply node.
= 1 failed, 664 passed, 29 skipped, 22 xfailed, 55 warnings in 1363.94s (0:22:43) =
 ** On entry to DGEES  parameter number  6 had an illegal value

[jessegrabowski]

No, but looks like an overly strict check somewhere? Like its doing if any(dim <= 0) to check for negativity. I can try to dig in and see if its an upstream version issue but it'll take me a second to get to.

[ricardoV94]

Maybe a new version of scipy thing?

[ricardoV94]

I haven't seen it fail in other CI. Does it have random numbers in it?

[Armavica]

You are sometimes adding/renaming a fourth argument with different names: eval_points, output_grads, g_outs: it sometimes looks like a missed opportunity to directly call them co|tangents as in the function prototypes!
If you don't feel like regenerating the PR I'm happy to clean it up afterwards (it is part of what I was doing in #1792 before the renaming)

[ricardoV94]

You are sometimes adding/renaming a fourth argument with different names: eval_points, output_grads, g_outs: it sometimes looks like a missed opportunity to directly call them co|tangents as in the function prototypes!
If you don't feel like regenerating the PR I'm happy to clean it up afterwards (it is part of what I was doing in #1792 before the renaming)

If this sounds good to go otherwise then I would merge and gladly take your cleanup after

[jessegrabowski]

wanna depreciate the grad method while we're at it?