Skip to content

Conversation

@MarkDaoust
Copy link
Member

No description provided.

@MarkDaoust MarkDaoust requested a review from a team as a code owner March 11, 2024 19:56
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added status:awaiting review PR awaiting review from a maintainer component:documentation Update docs labels Mar 11, 2024
@MarkDaoust
Copy link
Member Author

@mdda

"source": [
"Import the necessary packages."
]
},
Copy link
Contributor

@pcoet pcoet Mar 11, 2024

Choose a reason for hiding this comment

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

couple of typos here: "only task also accpets" -> "only task that also accepts"


Reply via ReviewNB

@pcoet pcoet assigned pcoet and unassigned pcoet Mar 11, 2024
"source": [
"Import the necessary packages."
]
},
Copy link
Member

@markmcd markmcd Mar 12, 2024

Choose a reason for hiding this comment

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

Can we say more than "affects the embedding"?

e.g. something like "Separating the title from the content ensures your content is modeled correctly when calculating similarity scores." and also "To ensure reliable results, include or omit the title consistently across your corpus."

@sidgoogle gave me a quick review on these claims :)


Reply via ReviewNB

@MarkDaoust
Copy link
Member Author

Friendly ping.

Copy link
Contributor

pcoet commented Mar 29, 2024

I'll approve to unblock you, but did you see the comments?

@joefernandez
Copy link
Contributor

We've been sitting on this for 12 months, so probably obsolete. Closing this to clean up the queue.

@github-actions github-actions bot removed the status:awaiting review PR awaiting review from a maintainer label Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants