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

chore: port the remaining doctoral course #247

Merged
merged 5 commits into from
Aug 20, 2024
Merged

Conversation

Saransh-cpp
Copy link
Member

@Saransh-cpp Saransh-cpp commented Aug 15, 2024

Closes #238

Related to "Research Data in Python"

@Saransh-cpp Saransh-cpp marked this pull request as draft August 15, 2024 13:11
@Saransh-cpp Saransh-cpp force-pushed the saransh/port-doctoral-2 branch 2 times, most recently from aa4f47c to b7f220c Compare August 19, 2024 13:58
ch02data/061internet.ipynb.py Outdated Show resolved Hide resolved
ch02data/066QuakeExercise.ipynb.py Outdated Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Outdated Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Outdated Show resolved Hide resolved
@Saransh-cpp Saransh-cpp force-pushed the saransh/port-doctoral-2 branch from 622c965 to 23758d9 Compare August 19, 2024 15:14
@Saransh-cpp Saransh-cpp force-pushed the saransh/port-doctoral-2 branch from 6f3c599 to 3bfebc5 Compare August 19, 2024 15:19
@Saransh-cpp Saransh-cpp marked this pull request as ready for review August 19, 2024 15:19
@Saransh-cpp Saransh-cpp requested a review from dpshelio August 19, 2024 15:20
Copy link
Member

@dpshelio dpshelio left a comment

Choose a reason for hiding this comment

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

Perfect! Some small things to update.

ch02data/061internet.ipynb.py Show resolved Hide resolved
ch02data/062basic_plotting.ipynb.py Outdated Show resolved Hide resolved
ch02data/062basic_plotting.ipynb.py Outdated Show resolved Hide resolved
ch02data/062basic_plotting.ipynb.py Outdated Show resolved Hide resolved
ch02data/066QuakeExercise.ipynb.py Show resolved Hide resolved
ch02data/068QuakesSolution.ipynb.py Outdated Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Outdated Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Show resolved Hide resolved
ch02data/082NumPy.ipynb.py Outdated Show resolved Hide resolved
@Saransh-cpp Saransh-cpp force-pushed the saransh/port-doctoral-2 branch from 7a3ca78 to dcd4a3a Compare August 20, 2024 10:16
@Saransh-cpp
Copy link
Member Author

Thanks for the review, @dpshelio! All the suggestions should be in now 🚀

@Saransh-cpp Saransh-cpp requested a review from dpshelio August 20, 2024 10:18
Copy link
Member

@dpshelio dpshelio left a comment

Choose a reason for hiding this comment

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

🙌

@dpshelio dpshelio merged commit d78d9d9 into main Aug 20, 2024
5 checks passed
@dpshelio dpshelio deleted the saransh/port-doctoral-2 branch August 20, 2024 16:52
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.

Bring descriptions from doctoral school material
2 participants