Skip to content

Add overview of geoscience stack #265

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

Merged
merged 23 commits into from
Apr 20, 2022

Conversation

DamienIrving
Copy link
Contributor

I've had a go at expanding the overview of the Python geoscience software stack.

The content I've added has been adapted from this page on the PyAOS website. Incorporating some of the PyAOS content into Pythia Foundations is something we've been discussing for a while.

@DamienIrving DamienIrving requested a review from a team as a code owner April 18, 2022 01:30
@DamienIrving DamienIrving requested review from brian-rose and mgrover1 and removed request for a team April 18, 2022 01:30
@github-actions
Copy link

github-actions bot commented Apr 18, 2022

This pull request is being automatically built with GitHub Actions and Netlify. To see the status of your deployment, click below.

🔍 Git commit SHA: 4e90cd2
✅ Deployment Preview URL: https://626005380d896f20b0f8f21f--pythia-foundations.netlify.app

Copy link
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

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

Hi @DamienIrving, thanks for getting this started!

I just went through quickly and added several suggested edits to use consistent capitalization of library names.

I think this is a really nice starting point for our presentation of the stack.

Copy link
Contributor

@mgrover1 mgrover1 left a comment

Choose a reason for hiding this comment

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

Thanks for taking the time to put together this fantastic overview @DamienIrving - just a couple of minor comments. This will be a great addition to the Foundations content 😁

DamienIrving and others added 19 commits April 19, 2022 07:08
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
@DamienIrving
Copy link
Contributor Author

Thanks, @brian-rose and @mgrover1. I've gone ahead and made all the changes you suggested.

Copy link
Contributor

@mgrover1 mgrover1 left a comment

Choose a reason for hiding this comment

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

Thanks for making those changes @DamienIrving !

Copy link
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

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

Sorry just one more suggestion, with an explicit link to the Pythia Resource Gallery. Otherwise good to go!

Co-authored-by: Brian Rose <[email protected]>
@DamienIrving
Copy link
Contributor Author

Done :-)

Copy link
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

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

Fantastic, thanks @DamienIrving

@brian-rose brian-rose merged commit 6c8f8d1 into ProjectPythia:main Apr 20, 2022
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