Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BLOG] The Convoluted Story Behind np.top_k #868

Merged

Conversation

JuliaPoo
Copy link
Contributor

@JuliaPoo JuliaPoo commented Jul 17, 2024

Text styling

  • The blog is written with plain language (where relevant).
  • If there are headers, they use the proper header tags in order to do so (with only one level-one header).
  • All links describe where they link to (for example, check the Quansight labs website).
  • Any kind of styling that the author uses (for example, bold for emphasis) is consistent throughout the blog.

Non-text contents

  • Blog post featured image is in PNG or JPEG format, not SVG.
  • All content is represented as text (for example, images need alt text and videos need captions or descriptive transcripts).
  • If there are emojis, there are not more than three in a row.
  • Don't use flashing gifs or videos.
  • If it were to be read as plain text, the blog still makes sense and no information is missing.

Copy link

vercel bot commented Jul 17, 2024

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

Name Status Preview Comments Updated (UTC)
labs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 19, 2024 8:08am

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

This is a very nice blog post, thanks @JuliaPoo! The story line works quite well I think. I have only two content-related comments, and a handful of minor textual ones.

apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
apps/labs/posts/convoluted-story-behind-topk.md Outdated Show resolved Hide resolved
@rgommers rgommers added labs 🔭 Items related to the Labs website type: content 📝 labels Jul 18, 2024
@JuliaPoo
Copy link
Contributor Author

I've addressed the comments!

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

This looks great, and is ready to go. Hitting the green button, thanks again Jules!

@rgommers rgommers merged commit abd5f54 into Quansight:main Jul 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
labs 🔭 Items related to the Labs website type: content 📝
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants