Skip to content

Commit b614322

Browse files
committed
Removed content_hash wipe if only one is present, as this is overly stateful to __add__
1 parent e9621a9 commit b614322

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

src/paperqa/types.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1038,13 +1038,11 @@ def __add__(self, other: DocDetails | int) -> DocDetails: # noqa: PLR0912
10381038
)
10391039
else:
10401040
merged_data[field] = max(self_value, other_value)
1041-
elif field == "content_hash" and ( # noqa: PLR0916
1042-
# Hashes are both present but differ
1043-
(self_value and other_value and self_value != other_value)
1044-
# One hash is explicitly disabled (not autopopulated)
1045-
or (self_value is None or other_value is None)
1041+
elif field == "content_hash" and (
1042+
self_value and other_value and self_value != other_value
10461043
):
1047-
# We don't know which to pick, so just discard the value
1044+
# If hashes are both present but differ,
1045+
# we don't know which to pick, so just discard the value
10481046
merged_data[field] = None
10491047

10501048
else:

0 commit comments

Comments
 (0)