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

Add geovista to external examples #3558

Merged
merged 1 commit into from
Nov 9, 2022
Merged

Conversation

akaszynski
Copy link
Member

Discovered geovista from looking at @banesullivan's forked respositories and thought it would be a great addition to our external examples page.

@bjlittle, do you mind if we include https://raw.githubusercontent.com/bjlittle/geovista-data/main/media/oisst-avhrr.png and a link to your GitHub repo in our External Examples?

@github-actions github-actions bot added the documentation Anything related to the documentation/website label Nov 5, 2022
@banesullivan
Copy link
Member

I have some other screenshots from GeoVista too. An excellent library!

lfric_sst
oisst_avhrr_sst
ww3_global_tri

@codecov
Copy link

codecov bot commented Nov 5, 2022

Codecov Report

Merging #3558 (aaf4a33) into main (74298f3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3558   +/-   ##
=======================================
  Coverage   95.16%   95.16%           
=======================================
  Files          83       83           
  Lines       18571    18571           
=======================================
  Hits        17674    17674           
  Misses        897      897           

Copy link
Member

@tkoyama010 tkoyama010 left a comment

Choose a reason for hiding this comment

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

LGTM. Cool library!

@akaszynski
Copy link
Member Author

Rolling with this. We can always take this down later.

@akaszynski akaszynski merged commit be791c2 into main Nov 9, 2022
@akaszynski akaszynski deleted the docs/add-geo-vista-external branch November 9, 2022 17:36
@banesullivan banesullivan mentioned this pull request Feb 1, 2023
5 tasks
@bjlittle
Copy link
Contributor

@akaszynski @banesullivan @tkoyama010

Sorry, I'm late to the party (story of my life 🤣)

Thanks so much 👍

Totally humbled that you guys have added this to your external examples.

PyVista FTW 🍻

@bjlittle
Copy link
Contributor

bjlittle commented Apr 26, 2023

@akaszynski @banesullivan @tkoyama010

BTW I'm attending SciPy 2023 Conference this year, July 12-14.

All booked up, and good to go. Can't wait.

I'm intending to do a lightning talk on geovista and generally sing the praises of pyvista.

Are any of you guys attending? I'd totally love to catch up with any pyvista core devs that'll be around. I'll also be there for the sprinting at the weekend, July 15-16, if anyone is up for a slice of that?

@tkoyama010
Copy link
Member

tkoyama010 commented Apr 26, 2023

@bjlittle We haven't officially announced it yet, but the PyVista community will be participating in Scipy tutorials, poster sessions, and sprints! Check the program later. All three of us here are planning to attend and look forward to seeing you. We're looking forward to seeing you there. geovista is doing a great job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Anything related to the documentation/website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants