Python 3.11 issue with productions.py: inspect.getargspec() -> inspect.getfullargspec()

[NoTime4Inertia]

In line 21 and line 61 of productions.py, inspect.getargspec() is depreciated in Python 3.11. It must be changed to inspect.getfullargspec(); however, I think a different error results from that change, and I haven't had the chance to review it yet.

[asmaloney]

I have a commit from a couple of years ago that fixed this on the previous incarnation of this repository.

It can probably be updated and applied if there's anybody still working on this project.

[asmaloney]

If you need to use Python 3.11+, my fork of python_actr seems to be working for my use cases.

Here are the changes required to fix this getargspec issue.

[rlwest]

Thanks for these I’m travelling to the middle of the month but I will push all of them!! cheers R

…

On Aug 4, 2023, at 12:02 PM, Andy Maloney ***@***.***> wrote: If you need to use Python 3.11+, my fork of python_actr <https://github.com/asmaloney/python_actr> seems to be working for my use cases. Here <asmaloney@eb0fb8d> are the changes required to fix this getargspec issue. — Reply to this email directly, view it on GitHub <#19 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABR4VBIIBTIUGNYRQBX4ZZLXTVBMFANCNFSM6AAAAAAUJIM4L4>. You are receiving this because you are subscribed to this thread.

[asmaloney]

OK - if you are going to merge PRs, I'll make one for this issue.

[asmaloney]

@rlwest

I’m travelling to the middle of the month but I will push all of them!!

BTW - there's no "push" involved here. You just merge the PRs using the interface here on GitHub. If you go to a PR there's a green "Merge" (or similarly-named) button.