Reputation: 4516
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?
Upvotes: 3
Views: 878
Reputation: 106936
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
Upvotes: 3
Reputation: 1982
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
Upvotes: 0