By the way, skip_bound_arg parameter name is a little bit long. Maybe you can rename it to bound_arg.

@pitrou: Do you have an opinion on the parameter name? :-)

I would prefer to have skip_bound_arg as a name, because it has a history (from existing code in inspect.py):

skip_bound_arg=False is a double negation, and I'm always confused by double negation :-( bound_arg=True is more straightfoward to me.

Previously, the argument was private, no?

Yes, it was private. It only existed on protected _signature_from_callable helper.

Done, now it is named bound_arg! 👍

What's the motivation for this change? I can't find the discussion behind it.

My worry is that it only works on some callables, and it's not very clear which ones those are -- or how to make a custom callable (e.g. one implemented in the C API) work like a method.

What's the motivation for this change?

I want to deprecate getfullargspec, which works almost like signature(), but with this difference. See

My worry is that it only works on some callables, and it's not very clear which ones those are -- or how to make a custom callable (e.g. one implemented in the C API) work like a method.

It will work exactly like this older function.

Is this behaviour useful, or is it a bug that's kept in getfullargspec for backwards compatibility?

Is this behaviour useful, or is it a bug that's kept in getfullargspec for backwards compatibility?

IMO it's useful in general, not only for backward compatibility. Keep self or not should be configurable.

Is this behaviour useful, or is it a bug that's kept in getfullargspec for backwards compatibility?

I think it was primarily a quirk of the way getfullargspec was originally implemented (i.e. it followed bound methods down to the underlying function to get the signature details, and then just returned those details as is without accounting for the first parameter already being bound).

However, the suggested flag isn't hard to maintain, since the complexity is in the branch that corrects the underlying function signature to remove the already bound first parameter (the default behaviour). The "unwrap bound methods" branch is literally just return sig:

I do think making the flag public is worth including as a step prior to full programmatic deprecation of getfullargspec. With the suggested docs change, I also think the bound_arg parameter name is a reasonable one.

(The one potential argument I see for a parameter name like unbind_methods over bound_arg, is that the flag specifically affects the handling of the bound arg in bound methods. It doesn't have any effect on the handling of the previously bound arguments in functools.partial and functools.partialmethod instances. However, I don't think that concern is significant enough to be worth changing the parameter name again)

the suggested flag isn't hard to maintain

My worry is that while the flag isn't hard to maintain as an internal detail of how getfullargspec extracts information from Python functions/methods, signature should work for other callables, like instances of C extension types (e.g. ones with Py_TPFLAGS_METHOD_DESCRIPTOR), or Cython functions. We might not be able to support these things transparently, but there should be a clear way to make custom objects that work with the flag. Or a documented way, at least.

(Unfortunately I'll probably not have time to investigate this worry before beta 1. If the feature is waiting for my review, it won't make it.)

This PR is stale because it has been open for 30 days with no activity.