3

I would like to merge two dictionaries A and B, knowing a common previous state C of both dictionaries. I need the merge to occur in subdictionaries too. In the case of real conflict, I need an exception to be raised.

1 - In the following example, the merge method should understand that A and B edited different items, and thus the merge should not raise a conflict

C = {"x": 0, "y": 0}
A = {"x": 1, "y": 0} # Edit x, but not y
B = {"x": 0, "y": 1} # Edit y, but not x
# merge(A, B, C) => {"x": 1, "y": 1}

2 - The function needs to be able to deal with new items and deleted items

C = {"x": 0}
A = {"x": 0, "y": 0} # Add y, keep x untouched
B = {}               # Delete x
# merge(A, B, C) => {"y": 0}

3 - The function should raise an exception when a real conflict occurs

C = {"x": 0}
A = {"x": 1}         # Edit x 
B = {"x": 2}         # Also edit x
# merge(A, B, C) => raise Exception

C = {"x": 0}
A = {"x": 1}         # Edit x 
B = {}               # Delete x
# merge(A, B, C) => raise Exception

4 - The function should work recursively

C = {"deeper": {"x": 0, "y": 0}}
A = {"deeper": {"x": 1, "y": 0}} # Edit deeper["x"], but not deeper["y"]
B = {"deeper": {"x": 0, "y": 1}} # Edit deeper["y"], but not deeper["x"]
# merge(A, B, C) => {"deeper": {"x": 1, "y": 1}}

What's the best approach to implement such a merge function?

azmeuk
  • 2,860
  • 3
  • 22
  • 50

2 Answers2

3

You can convert all the dict items into sets, use the intersection of keys from the symmetric differences to C to find conflicts, and use a union of the 3 sets' intersection (common items) and the differences to C to obtain the merge. Recursively merge sub-dicts that are common to A, B and C, convert sub-dicts into tuples of item pairs to allow them to be hashable and convertible into sets, and then convert them back to dicts after merge.

EDIT: In case the dict values are unhashable objects such as a set, you would have to serialize the values (I recommend using pickle as a serializer since it has native support from Python) before you can convert the dict items into a set, and de-serialize them after the merge:

import pickle

def merge(a, b, c):
    # recursively merge sub-dicts that are common to a, b and c
    for k in a.keys() & b.keys() & c.keys():
        if all(isinstance(d.get(k), dict) for d in (a, b, c)):
            a[k] = b[k] = c[k] = merge(a[k], b[k], c[k])
    # convert sub-dicts into tuples of item pairs to allow them to be hashable
    for d in a, b, c:
        for k, v in d.items():
            if isinstance(v, dict):
                d[k] = tuple(v.items())
    # convert all the dict items into sets
    set_a, set_b, set_c = (set((k, pickle.dumps(v)) for k, v in d.items()) for d in (a, b, c))
    # intersect keys from the symmetric set differences to c to find conflicts
    for k in set(k for k, _ in set_a ^ set_c) & set(k for k, _ in set_b ^ set_c):
        # it isn't really a conflict if the new values of a and b are the same
        if a.get(k) != b.get(k) or (k in a) ^ (k in b):
            raise ValueError("Conflict found in key %s" % k)
    # merge the dicts by union'ing the differences to c with the common items
    d = dict(set_a & set_b & set_c | set_a - set_c | set_b - set_c)
    # convert the tuple of items back to dicts for output
    for k, v in d.items():
        v = pickle.loads(v)
        if isinstance(v, tuple):
            d[k] = dict(v)
        else:
            d[k] = v
    return d

so that:

C = {"x": 0, "y": 0}
A = {"x": 1, "y": 0} # Edit x, but not y
B = {"x": 0, "y": 1} # Edit y, but not x
print(merge(A, B, C))
C = {"x": 0}
A = {"x": 0, "y": 0} # Add y, keep x untouched
B = {}               # Delete x
print(merge(A, B, C))
C = {"x": 0}
A = {"x": 1}  # Edit x
B = {"x": 1}  # Edit x with the same value
print(merge(A, B, C))
C = {"deeper": {"x": 0, "y": {3, 4}}}
A = {"deeper": {"x": {1, 2}, "y": {4, 3}}} # Edit deeper["x"], but not deeper["y"]
B = {"deeper": {"x": 0, "y": 1}} # Edit deeper["y"], but not deeper["x"]
print(merge(A, B, C))
C = {"deeper": 1}
A = {"deeper": {"x": 0, "y": 1}} # Edit deeper and turn it into a dict
B = {"deeper": 1, "x": 2} # Add x, keep deeper untouched
print(merge(A, B, C))
C = {"deeper": {"x": 0, "y": 1}}
A = {"deeper": {"x": 0, "y": 1}} # Keep deeper untouched
B = {"deeper": 1} # Turn deeper into a scalar
print(merge(A, B, C))

would output:

{'x': 1, 'y': 1}
{'y': 0}
{'x': 1}
{'deeper': {'x': {1, 2}, 'y': 1}}
{'deeper': {'x': 0, 'y': 1}, 'x': 2}
{'deeper': 1}

while:

C = {"x": 0}
A = {"x": 1}         # Edit x
B = {"x": 2}         # Edit x with a different value
print(merge(A, B, C))

would raise:

ValueError: Conflict found in key x

and:

C = {"deeper": {"x": 0, "y": 1}}
A = {"deeper": {"x": 0, "y": 2}} # Edit deeper["y"], but not deeper["x"]
B = {"deeper": 1} # Turn deeper into a scalar
print(merge(A, B, C))

would raise:

ValueError: Conflict found in key deeper
blhsing
  • 70,627
  • 6
  • 41
  • 76
  • Nice! It does not raise an exception with `C = {"x": 0} A = {} B = {"x": None}`. It is because of `a.get(k) != b.get(k)` I think. – azmeuk Oct 11 '18 at 12:25
  • Indeed. Fixed with an additional test with the XOR operator then. – blhsing Oct 11 '18 at 12:34
  • One year after, I found a limitation to this solution. If the dict values are not hashable, the line `set_a, set_b, set_c = (set(d.items()) for d in (a, b, c))` fails. – azmeuk Aug 31 '19 at 08:14
0

try a function that recursively checks your cases and fails (raises a valueerror) if none of them are met.

I believe this satisfies your cases

def three_way_merge(Origin, Dict_A, Dict_B):
    newdict = dict()
    for key, value in Origin.items():
        if isinstance(value, dict):  # handle recursion
            newdict[key] = three_way_merge(Origin[key], Dict_A[key], Dict_B[key])
        elif key not in Dict_A.keys() and Dict_B[key] == value:
            pass
        elif key not in Dict_B.keys() and Dict_A[key] == value:
            pass
        elif Dict_A[key] == value and Dict_B[key] == value:
             newdict[key] = value
        elif Dict_A[key] == value and Dict_B[key] != value:
             newdict[key] = Dict_B[key]
        elif Dict_A[key] != value and Dict_B[key] == value:
             newdict[key] = Dict_A[key]
        elif Dict_A[key] == Dict_B[key]:
             newdict[key] = Dict_A[key]
        else: # check for a conflict with this key
            raise ValueError('conflict occured with {} \n {} and {} both changed'.format(key, Dict_A[key], Dict_B[key]))
    newdict.update(add_missing_keys(Dict_A, Origin, Dict_B))
    newdict.update(add_missing_keys(Dict_B, Origin, Dict_A))
    return newdict    

def add_missing_keys (Dict_A, Origin, Dict_B):
    newdict = dict()
    for key, value in Dict_A.items():
        if key not in Origin.keys():
            if key not in Dict_B.keys() or Dict_B[key] == value:
                newdict[key] = value
            else:
                raise ValueError('conflict occured with {} \n {} and {} both changed'.format(key, Dict_A[key], Dict_B[key]))
    return newdict

print(three_way_merge({'x':0, 'y':0}, {'x':1, 'y':0}, {'x':0, 'y':2}))     # returns {'x':1, 'y':2}
print(three_way_merge({'x':0}, {'x':0, 'y':0}, {}))    # returns {'y':0}
print(three_way_merge({'x':0}, {'x':1}, {'x':1}))    # returns {'x':1}
print(three_way_merge({"deeper": {"x": 0, "y": 0}},{"deeper": {"x": 1, "y": 0}},{"deeper": {"x": 0, "y": 2}})) # returns {'deeper': {'x': 1, 'y': 2}}
print(three_way_merge({'x':0}, {'x':1}, {'x':2})) # raises valueerror
print(three_way_merge({'x':0}, {'x':1}, {}))  # raises keyerror
Marc Wagner
  • 912
  • 1
  • 7
  • 15
  • This would not work if a value in the origin dict is a sub-dict and one of the new dicts tries to edit it into a scalar value. For example, `three_way_merge({'x':0, 'y': {'a': 1}}, {'x':1, 'y':0}, {'x':0, 'y': {'a': 1}})` would raise `AttributeError: 'int' object has no attribute 'keys'`. – blhsing Oct 11 '18 at 11:45
  • Not sure if that is unexpected behavior. Op did not mention changing types, or overwriting subdicts. – Marc Wagner Oct 11 '18 at 19:55