Skip to content

Commit 1df5ece

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

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
@@ -1133,13 +1133,11 @@ def __add__(self, other: DocDetails | int) -> DocDetails: # noqa: PLR0912
11331133
)
11341134
else:
11351135
merged_data[field] = max(self_value, other_value)
1136-
elif field == "content_hash" and ( # noqa: PLR0916
1137-
# Hashes are both present but differ
1138-
(self_value and other_value and self_value != other_value)
1139-
# One hash is explicitly disabled (not autopopulated)
1140-
or (self_value is None or other_value is None)
1136+
elif field == "content_hash" and (
1137+
self_value and other_value and self_value != other_value
11411138
):
1142-
# We don't know which to pick, so just discard the value
1139+
# If hashes are both present but differ,
1140+
# we don't know which to pick, so just discard the value
11431141
merged_data[field] = None
11441142

11451143
else:

0 commit comments

Comments
 (0)