Actually, I'd like to see a specific use case for this change. Either I don't understand something, or the only thing Python.NET does not support currently is passing Python callables toAction<*> parameters. Reverse works though.

So you only actually need to implementIPyObjectDecoder.

Actually, I'd like to see a specific use case for this change. Either I don't understand something, or the only thing Python.NET does not support currently is passing Python callables toAction<*> parameters. Reverse works though.

So you only actually need to implementIPyObjectDecoder.

@lostmsu I need to write an embedding test to show what I want to do.

@lostmsu I found that getting the number of arguments in python using inspect signature is not reliable because in the CanDecode method the codec only has the PyObject_TYPE of the PyObject rather than the object itself. Are you open to changing the contract of the codecs to pass in the object itself and let the codec get its type?

I used delegate reflection to get the number of arguments from the C# delegate. But I don't see how that would help with binding to any delegate types. I need to construct a delegate of the type given by the parameter argument and then return it from the decoder. I guess I can use four delegate types:

delegate void UnaryActionDelegate();
delegate void VariadicActionDelegate(object[] args);
delegate object UnaryFuncDelegate();
delegate object VariadicFuncDelegate(object[] args);

which I can easily construct. But you cannot even construct a System.Action with a UnaryActionDelegate instance. So I am failing to see how the delegate reflection will help with supporting Action and Func.

That being said I think it makes sense to implement these delegates anyways so I can support delegate binding.

@lostmsu I am working on moving this codec to the codecs repo. While doing so I am encountering some build issues related to access levels in the core library:

• Runtime.XIncref is marked internal
• I can't use Converter.ToManaged or ToPython because the Converter class is internal

We can fix it by setting InternalsVisibleTo in Python.Runtime to the codecs assembly, but this will not fix things for folks (like myself) who may want to implement codecs in my own external libraries. Are there alternatives to these methods that are public and if not what options would I have?

@koubaa do we have to? Can you achieve the same using publicPyObject class?

@lostmsu I don't see a way to increment a reference count using PyObject or to use Converter.ToManaged.

@koubaa instead of incrementing refcount you can just keep the PyObject.PyObject.As<T> is a direct counterpart ofConverter.ToManaged.

@lostmsu I'll try it.

@lostmsu that was helpful. I can use As<> and even object.ToPython (a public extension method defined in python runtime). Testing will show if I get the reference count working, will update the codecs repo soon