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.

Sign up for GitHub

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

Jump to bottom

Add kvikio blogpost #308

Merged
merged 26 commits into from
Aug 30, 2022
Merged

Add kvikio blogpost #308

merged 26 commits into from
Aug 30, 2022

Conversation

dcherian
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Aug 25, 2022

@dcherian is attempting to deploy a commit to the xarray Team on Vercel.

A member of the Team first needs to authorize it.

@dcherian dcherian marked this pull request as draft August 25, 2022 20:22
@dcherian
Copy link
Contributor Author

@jakirkham @weiji14 Reviews / contributions would be very helpful!

dcherian
dcherian commented Aug 25, 2022
View reviewed changes
src/posts/xarray-kvikio/index.md Outdated

## Cool demo

We don't have a cool demo yet but are looking to develop one very soon!
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Link to xarray-contrib/xbatcher#87?

Copy link
Member

Choose a reason for hiding this comment

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

Sure, and/or we can link to the forum at https://discourse.pangeo.io/tag/machine-learning if people want to reach out.

@dcherian dcherian mentioned this pull request Aug 25, 2022
Open
@vercel
Copy link

vercel bot commented Aug 25, 2022 •
edited
Loading

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

Name Status Preview Updated
xarray-dev ✅ Ready (Inspect) Visit Preview Aug 30, 2022 at 3:32PM (UTC)

weiji14
weiji14 reviewed Aug 25, 2022
View reviewed changes
Copy link
Member

@weiji14 weiji14 left a comment

Choose a reason for hiding this comment

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

Nice work @dcherian, you make it look so easy 😆 But really, there was a mountain of work that had to happen behind the scenes to get to this point. Just a few minor suggestions on the draft for now 😃

src/posts/xarray-kvikio/index.md Outdated

## Cool demo

We don't have a cool demo yet but are looking to develop one very soon!
Copy link
Member

Choose a reason for hiding this comment

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

Sure, and/or we can link to the forum at https://discourse.pangeo.io/tag/machine-learning if people want to reach out.

dcherian and others added 4 commits August 25, 2022 15:15
@dcherian @weiji14
Update src/posts/xarray-kvikio/index.md
c1b09c2 
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@dcherian @weiji14
Update src/posts/xarray-kvikio/index.md
804c262 
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@dcherian @weiji14
Update src/posts/xarray-kvikio/index.md
4a09f34 
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@dcherian @weiji14
Apply suggestions from code review
0f579a6 
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@dcherian @andersy005
Update src/posts/xarray-kvikio/index.md
56589f9 
Co-authored-by: Anderson Banihirwe <axbanihirwe@ualr.edu>
weiji14
weiji14 reviewed Aug 26, 2022
View reviewed changes
dcherian and others added 2 commits August 26, 2022 10:19
@dcherian @weiji14
Apply suggestions from code review
dbd8880 
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@dcherian @weiji14
Apply suggestions from code review
a709f41 
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@dcherian @andersy005
Update src/posts/xarray-kvikio/index.md
eb2c4d9 
Co-authored-by: Anderson Banihirwe <axbanihirwe@ualr.edu>
@dcherian dcherian marked this pull request as ready for review August 26, 2022 17:40
@dcherian
Copy link
Contributor Author

dcherian commented Aug 26, 2022 •
edited
Loading

I'm done here and heading out on vacation for two weeks. Feel free to merge after review and tweet about it :)

weiji14
weiji14 reviewed Aug 29, 2022
View reviewed changes
src/posts/xarray-kvikio/index.md Outdated
@@ -0,0 +1,178 @@
---
title: 'Enabling GPU-native analytics with Xarray and kvikIO'
date: '2022-08-25'
Copy link
Member

Choose a reason for hiding this comment

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

Need to update this date before publishing.

Do we want to publish the blog post soon-ish (e.g. by end of August) or wait for a bit longer (e.g. until some of those WIP PRs get a bit more stable)? Btw, I don't have merge permissions on this repo...

Copy link
Member

Choose a reason for hiding this comment

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

@weiji14, i updated the date. Regarding whether we should wait or ship this as is, i'll defer to you :)

Copy link
Member

Choose a reason for hiding this comment

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

Let's ship it 🚀

@andersy005 andersy005 enabled auto-merge (squash) August 30, 2022 15:31
@andersy005 andersy005 merged commit c72070b into xarray-contrib:main Aug 30, 2022
@weiji14
Copy link
Member

weiji14 commented Aug 30, 2022 •
edited
Loading

Cool, thanks @andersy005! Blog post is live on https://xarray.dev/blog/xarray-kvikio 🥳 I'll sent out a tweet shortly :D

Edit: Tweet at https://twitter.com/weiji14_/status/1564647885467181056

@jakirkham
Copy link

Thanks all! This looks great 😄

cc @madsbk @jacobtomlinson @quasiben

@dcherian dcherian deleted the kvikio-post branch September 9, 2022 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andersy005 andersy005 andersy005 left review comments

@weiji14 weiji14 weiji14 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dcherian @weiji14 @jakirkham @andersy005