Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IndexError: tuple index out of range #322

Open
dustinbyrne opened this issue May 25, 2024 · 1 comment
Open

IndexError: tuple index out of range #322

dustinbyrne opened this issue May 25, 2024 · 1 comment

Comments

@dustinbyrne
Copy link
Contributor

Found in sympy v1.0 test cases.

Traceback (most recent call last):
  File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/utilities/runtests.py", line 1130, in wrapper
    test_func()
  File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/functions/special/tests/test_zeta_functions.py", line 95, in test_derivatives
    assert zeta(x, a).diff(x) == Derivative(zeta(x, a), x)
  File "/home/db/dev/applandinc/appmap-python/_appmap/wrapt/wrappers.py", line 712, in __call__
    return self._self_wrapper(self.__wrapped__, self._self_instance,
  File "/home/db/dev/applandinc/appmap-python/_appmap/instrument.py", line 129, in instrumented_fn
    return call_instrumented(f, instance, args, kwargs)
  File "/home/db/dev/applandinc/appmap-python/_appmap/instrument.py", line 92, in call_instrumented
    ret = f.fn(*args, **kwargs)
  File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/core/expr.py", line 2949, in diff
    return Derivative(self, *new_symbols, **assumptions)
  File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/core/function.py", line 1187, in __new__
    obj = expr._eval_derivative(v)
  File "/home/db/dev/applandinc/appmap-python/_appmap/wrapt/wrappers.py", line 712, in __call__
    return self._self_wrapper(self.__wrapped__, self._self_instance,
  File "/home/db/dev/applandinc/appmap-python/_appmap/instrument.py", line 129, in instrumented_fn
    return call_instrumented(f, instance, args, kwargs)
  File "/home/db/dev/applandinc/appmap-python/_appmap/instrument.py", line 92, in call_instrumented
    ret = f.fn(*args, **kwargs)
  File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/core/function.py", line 559, in _eval_derivative
    df = self.fdiff(i)
  File "/home/db/dev/applandinc/appmap-python/_appmap/wrapt/wrappers.py", line 712, in __call__
    return self._self_wrapper(self.__wrapped__, self._self_instance,
  File "/home/db/dev/applandinc/appmap-python/_appmap/instrument.py", line 129, in instrumented_fn
    return call_instrumented(f, instance, args, kwargs)
  File "/home/db/dev/applandinc/appmap-python/_appmap/instrument.py", line 103, in call_instrumented
    event.ExceptionEvent(
  File "/home/db/dev/applandinc/appmap-python/_appmap/event.py", line 509, in __init__
    {"class": fqname(class_), "message": str(exc), "object_id": id(exc)}
  File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/core/function.py", line 94, in __str__
    (self.args[1], self.args[0]))
IndexError: tuple index out of range
@apotterri
Copy link
Contributor

File "/tmp/tmpz56qw_4w/sympy__sympy__1.0/sympy/core/function.py", line 94, in str
This is in ArgumentIndexError.__str__, where ArgumentIndexError is defined as

class ArgumentIndexError(ValueError):

sympy 1.0 was testing python 2.6 through 3.5: https://github.com/sympy/sympy/blob/sympy-1.0/.travis.yml#L13 , but SWE-bench was using 3.9.

We should investigate whether using 3.9 was the root cause of this problem.

dustinbyrne referenced this issue in getappmap/appmap-js Aug 1, 2024
From `packages/navie-gh`: `node dist/index https://github.com/getappmap/appmap-python/issues/322`
After some thinking an analysis will open in VS Code. There are two key
elements I wanted to explore:
1. Navie as an automated process, via `git clone` and some line of
   prompting to accomplish a task in the repository, like helping with
   an issue.
2. Navie as a programmatic API without a CLI dependency. This is a super
   sloppy copy and paste to move client behavior out of the CLI. It's
   currently duplicated, but gives a sense of what it'd take to extract
   as a separate library. We're probably better off with the CLI
   dependency for now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants