Skip to content

Enable user classes to override how Python.NET processes parameters of their functions #835

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.

Sign up for GitHub

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

Closed
lostmsu wants to merge 11 commits into from
Closed

Enable user classes to override how Python.NET processes parameters of their functions #835

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
c355bd6
introduced IPyArgumentConverter interface, that controls marshaling o…
lostmsu Mar 28, 2019
b5cae6c
enabled an ability to override Python to .NET argument coversion usin…
lostmsu Mar 28, 2019
bea9e0d
mentioned PyArgConverter in the CHANGELOG.md
lostmsu Mar 28, 2019
11a3ee6
fixed pyargconverter.cs not included into the older project file
lostmsu Mar 29, 2019
9c81661
moved default arg converter to its own file; cached converter lookup
lostmsu Apr 14, 2019
b6421e9
fixed defaultpyargconverter.cs not included in the older project file
lostmsu Apr 17, 2019
6cd7bd4
when looking for a custom argument converter look at base types too
lostmsu Jun 13, 2019
56ca4c9
a test to ensure it is possible to override argument conversion for i…
lostmsu Jun 13, 2019
7f894fa
fixed MarshallerOverride test
lostmsu Jun 20, 2019
aaafea8
fixed crash in MethodBinder.Bind when MethodInfo to invoke is passed …
lostmsu Aug 22, 2019
3804438
Merge branch 'master' into PR/PyArgConverter
lostmsu Feb 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
a test to ensure it is possible to override argument conversion for i… 
…nherited methods using an attribute on a derived class
  • Loading branch information
@lostmsu
lostmsu committed Jan 31, 2020
commit 56ca4c995a0018ca48123bebfe4485b144a6f9de
39 changes: 34 additions & 5 deletions src/embed_tests/TestCustomArgMarshal.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,27 @@ public void CustomArgMarshaller()
}
Assert.AreEqual(expected: 42, actual: obj.LastArgument);
}

[Test]
public void MarshallerOverride() {
var obj = new DerivedMarshaling();
using (Py.GIL()) {
dynamic callWithInt = PythonEngine.Eval("lambda o: o.CallWithInt({ 'value': 42 })");
callWithInt(obj.ToPython());
}
Assert.AreEqual(expected: 42, actual: obj.LastArgument);
}
}

[PyArgConverter(typeof(CustomArgConverter))]
class CustomArgMarshaling {
public object LastArgument { get; private set; }
public void CallWithInt(int value) => this.LastArgument = value;
}
[PyArgConverter(typeof(CustomArgConverter))]
class CustomArgMarshaling {
public object LastArgument { get; protected set; }
public virtual void CallWithInt(int value) => this.LastArgument = value;
}

// this should override original custom marshaling behavior
[PyArgConverter(typeof(CustomArgConverter2))]
class DerivedMarshaling: CustomArgMarshaling { }

class CustomArgConverter : DefaultPyArgumentConverter {
public override bool TryConvertArgument(IntPtr pyarg, Type parameterType, bool needsResolution,
Expand All @@ -52,4 +66,19 @@ public override bool TryConvertArgument(IntPtr pyarg, Type parameterType, bool n
return true;
}
}

class CustomArgConverter2 : DefaultPyArgumentConverter {
public override bool TryConvertArgument(IntPtr pyarg, Type parameterType, bool needsResolution,
out object arg, out bool isOut) {
if (parameterType != typeof(int))
return base.TryConvertArgument(pyarg, parameterType, needsResolution, out arg, out isOut);
bool isPyObject = base.TryConvertArgument(pyarg, typeof(PyObject), needsResolution,
out arg, out isOut);
if (!isPyObject) return false;
var dict = new PyDict((PyObject)arg);
int number = (dynamic)dict["value"];
arg = number;
return true;
}
}
}