Skip to content

tweak module 5#21

Merged
PolarBean merged 4 commits into
mainfrom
module-5-tweaks
Jun 2, 2026
Merged

tweak module 5#21
PolarBean merged 4 commits into
mainfrom
module-5-tweaks

Conversation

@alessandrofelder
Copy link
Copy Markdown
Member

@alessandrofelder alessandrofelder commented Jun 1, 2026

What is this PR
Here is how I addressed each point in the issue

slide 95: niksirbi made a really nice NIU+Python ecosystem graphic that would go well here.

I put it here.

slide 97: mention environments here and say we will cover them later?

Minor rewording to keep envs and installing packages contained to module 8. I've linked to module 8 (implementing "open in new tab" behaviour" here and tested manually, but don't understand the code my AI-assistant generated for this.

slide 98: we should cover the concept of an array before we introduce numpy functions here. And maybe not use class, but refer back to datatype/objects explanations from earlier

Had a go at implementing this.

slide 99: should values be a np.array or is the solver expected to convert? (arguably you could/should use existing list functions for this exercise if you have a list) Can we introduce instantiation of arrays from a list and with things likes np.ones/zeros(_like) first?

Moved randint slide from module 7 here. And added slide and exercise about np.zeros and np.ones.

slide 100: "A NumPy array is a class useful for mathematical operations on an entire collection." I would maybe remove this sentence?

I removed this sentence.

slide 102: The answer includes the word "vectorised" which has not been introduced.

I reworded this sentence.

References

Closes #11 . Tested by visual inspection of local deployment, and clicking on the new hyperlink.

@alessandrofelder alessandrofelder marked this pull request as ready for review June 1, 2026 17:08
@PolarBean PolarBean merged commit ef5d000 into main Jun 2, 2026
@PolarBean PolarBean deleted the module-5-tweaks branch June 2, 2026 16:46
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.

Module 5 suggestions

2 participants