Skip to content

Conversation

@brunobelloni
Copy link

dataset is slightly smaller compared to the original example, but it works for educational purposes

@ted-at-openai ted-at-openai self-requested a review June 22, 2023 23:47
@ted-at-openai ted-at-openai self-assigned this Jun 22, 2023
@brunobelloni brunobelloni marked this pull request as draft June 24, 2023 00:54
@brunobelloni brunobelloni marked this pull request as ready for review June 24, 2023 00:55
@brunobelloni brunobelloni changed the title Dataset update User_and_product_embeddings.ipynb Dataset update Jun 28, 2023
@ted-at-openai
Copy link
Collaborator

Will take a look this week. Thanks for the contribution!

{
"cell_type": "markdown",
"metadata": {},
"cell_type": "raw",
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should stay as markdown

@ted-at-openai
Copy link
Collaborator

Thanks for the contribution.

Code seems to run, but text is now out of date.

Also, it looks like the conclusion (trend in scores) doesn't appear to hold in the smaller sample.

So I think I'll close this and we'll work on providing better, cleaned up instructions for the full dataset.

Thank you nonetheless.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants