Skip to content

Chapter 1 Review: update hyperlinks and outdated packages#187

Merged
jstac merged 13 commits intomainfrom
lec-review-1
Jul 29, 2022
Merged

Chapter 1 Review: update hyperlinks and outdated packages#187
jstac merged 13 commits intomainfrom
lec-review-1

Conversation

@HumphreyYang
Copy link
Copy Markdown
Member

Hi,

I have looked through the first chapter (About Python) of the lecture and found several outdated links.

Other than these, the only fix is a warning from iPython at line 324 due to a formatting issue.

Since this is a short chapter without too many changes made, so I put everything in one PR. Please kindly review the changes.

Thank you.

@jstac
Copy link
Copy Markdown
Contributor

jstac commented Jul 26, 2022

Great work @HumphreyYang , many thanks. This is very helpful!

Would you mind also reviewing the links to determine whether or not they are as up to date as possible?

For example, we cite Theano for GPUs but I don't think it's very popular any more. (I could be wrong but it seems to have been overtaken by JAX.)

I feel like we should at least mention JAX and PyTorch.

This is a harder task, but perhaps you could make one or two small changes along these lines as well?

@github-actions github-actions Bot temporarily deployed to commit July 27, 2022 00:44 Inactive
@jstac
Copy link
Copy Markdown
Contributor

jstac commented Jul 27, 2022

This is great, thanks @HumphreyYang .

Ready for us to merge once the checks have passed?

@github-actions github-actions Bot temporarily deployed to commit July 27, 2022 01:02 Inactive
@HumphreyYang
Copy link
Copy Markdown
Member Author

HumphreyYang commented Jul 27, 2022

Hi John,

I have just added some more on the recent development on HPC.

Here is a summary of what I have done in the latest commits:

  1. Added more recent packages in graphics, statistics, cloud computing, and high-performance computing (HPC);
  2. Provided more resources on data science and HPC;
  3. In the example section for Scipy, the book mentioned integral calculation, but there is no example for the integrate() method below. Thus, I added an example, a slightly complicated integral, in that section.

Please kindly check these recent updates, and leave a comment if there is anything more you would like me to add.

If everything looks good to you, please merge them whenever you think appropriate.

Thank you.

@github-actions github-actions Bot temporarily deployed to commit July 27, 2022 01:16 Inactive
@mmcky
Copy link
Copy Markdown
Contributor

mmcky commented Jul 27, 2022

thanks @HumphreyYang this is a nice update for the lecture.

@jstac
Copy link
Copy Markdown
Contributor

jstac commented Jul 27, 2022

Many thanks @HumphreyYang .

I think there are a few more changes we could make. For example,

@HumphreyYang
Copy link
Copy Markdown
Member Author

Hi John,

I think I have changed several points you mentioned in the latest deployment such as on
Seaborn and SageMath. But I will definitely review everything again and update the contents more aggressively.

Thank you so much for your feedback.

@jstac
Copy link
Copy Markdown
Contributor

jstac commented Jul 27, 2022

@HumphreyYang , sorry I missed those changes but yes please, one more round would be great.

@github-actions github-actions Bot temporarily deployed to commit July 29, 2022 01:23 Inactive
@HumphreyYang
Copy link
Copy Markdown
Member Author

HumphreyYang commented Jul 29, 2022

Hi @jstac and @mmcky,

Thank you so much for your feedback in the previous round. I started to grasp the required level of revision for the lectures.

In this round of revision, I incorporated the list of suggestions by @jstac and added more resources on the application of Python and libraries. I checked all listed libraries and made sure there are recent discussions on the library on StackOverflow.

The only suggestion that I failed to incorporate into this version is:

Although I failed to find solid support for Python as the most popular first language at the university level (a 2016 study in the UK is even against this statement), I did find many case studies on Python for students from non-CS backgrounds that demonstrate key advantages of Python.

Could you please kindly visit the latest employment and review these changes?

Many thanks.

@github-actions github-actions Bot temporarily deployed to commit July 29, 2022 12:44 Inactive
@jstac
Copy link
Copy Markdown
Contributor

jstac commented Jul 29, 2022

Hi @HumphreyYang , I've looked over this carefully and you've done a great job. I'm really happy to see this page up to date again! I'll go ahead and merge now.

@jstac jstac merged commit c364548 into main Jul 29, 2022
@jstac jstac deleted the lec-review-1 branch July 29, 2022 18:50
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