Deserialize Job Event Params

[wiwichips]

Previously, parameters passed to jon event handler callbacks weren't deserialized. See the example below:

example code:

#!/usr/bin/env python3

import dcp
dcp.init()

from dcp import compute_for

def workfn(datum):
    import dcp
    dcp.progress()

    my_dict = {
        'val': datum + datum
    }

    return my_dict

my_j = compute_for([1], workfn)

my_j.on('result', print)

my_j.public.name = 'simple bifrost2 example'
my_j.computeGroups = [{ 'joinKey': 'will', 'joinSecret': 'bozo' }]

my_j.exec()
res = my_j.wait()

Before this change, the output would be:

{'job': '0xec837b8Dbfc1eBac8A5C7bc51D7fb2Bd19B6286D', 'sliceNumber': 1.0, 'result': <memory at 0x73bb740e0280>}

After this change, the output is now:

{'job': '0x5ECfdb6a856221f070DE657344a5Fe5de8C14e6C', 'sliceNumber': 1.0, 'result': {'val': 2}}

the main diff is that now instead of <memory at 0x73bb740e0280> it prints {'val': 2}

[Xmader]

For readability, would it be better

Suggested change

	if isinstance(arg, dict) and bool(arg):
	if isinstance(arg, dict) and len(arg) > 0:

[wiwichips]

honestly it's a no-op anyways, not sure why I check the length. It could just be

if isinstance(arg, dict):

I'll change it to that

[wiwichips]

changed in dfe1623

[Xmader]

Do we need to change callback? If not, it's probably better to remove this line.

Suggested change

nonlocal callback