Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

small fix with entity #87

Merged
merged 1 commit into from
Jan 2, 2024
Merged

small fix with entity #87

merged 1 commit into from
Jan 2, 2024

Conversation

jordan-wu-97
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Jan 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
openassistants-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 2, 2024 7:13pm

@@ -35,8 +35,8 @@ def entity_to_document(entity: IEntity) -> Document:
metadata=dict(id=entity.get_identity()), page_content=entity.get_identity()
)

if entity.get_description:
doc.page_content += f" ({entity.get_description})"
if entity.get_description():
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good case for using the Walrus operator https://realpython.com/python-walrus-operator/

@jordan-wu-97 jordan-wu-97 merged commit 7875f4b into main Jan 2, 2024
4 checks passed
@jordan-wu-97 jordan-wu-97 deleted the jordan/small-entity-fix branch January 2, 2024 19:22
colemccracken pushed a commit that referenced this pull request Jan 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants