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 Python Programming and Numerical Methods #367

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ivanov
Copy link

@ivanov ivanov commented May 21, 2021

I came across this book while trying to find books using jupyter book (before I thought to look on this page). Though they have a GitHub link on their rendered version, it unfortunately just points back to the rendered page.

@welcome
Copy link

welcome bot commented May 21, 2021

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out EBP's Code of Conduct and our Contributing Guide, as this will greatly help the review process.

Welcome to the EBP community! 🎉

@chrisjsewell
Copy link
Member

Thanks @ivanov!

Though they have a GitHub link on their rendered version, it unfortunately just points back to the rendered page.

@choldgraf It says its from berkley, perhaps you have an idea who it could be. Would be nice to add the repository link before merging

@chrisjsewell chrisjsewell added the awaiting-response Awaiting a response to questions/review suggestions label Jun 3, 2021
choldgraf
choldgraf previously approved these changes Aug 21, 2022
Copy link
Member

@choldgraf choldgraf left a comment

Choose a reason for hiding this comment

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

Let's just merge this in and we can consider removing or requesting a github repository if the lack of a repo confuses people

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-response Awaiting a response to questions/review suggestions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants