Skip to content

Conversation

jstac
Copy link
Contributor

@jstac jstac commented Jul 26, 2025

Hi @mmcky , I've ported this over from the JAX GPU lecture series and tweaked it to change the focus to using JAX on CPUs.

Could you please read it carefully and make any necessary modifications -- or delegate this task? It's probably necessary to look at the text relating to timing of code execution and check that it still makes sense.

@mmcky
Copy link
Contributor

mmcky commented Jul 28, 2025

thanks @jstac

I will need to update the repo infrastructure to enable this series to access hardware with a gpu.

Tasks

  • @mmcky debug missing software and target jax running on cpu
  • @mmcky read the lecture carefully in the context of this lecture series.

@mmcky mmcky self-requested a review July 28, 2025 02:01
@mmcky mmcky self-assigned this Jul 28, 2025
@mmcky
Copy link
Contributor

mmcky commented Jul 28, 2025

@jstac just read the cpu target.

I will just have to update the software and debug. Thanks.

@jstac
Copy link
Contributor Author

jstac commented Jul 28, 2025

Thanks Matt. As you say, this is just for CPU.

JAX is already faster on the CPU so we can unify many of our lectures around this platform.

Copy link

github-actions bot commented Jul 31, 2025

@github-actions github-actions bot temporarily deployed to pull request July 31, 2025 04:55 Inactive
@mmcky
Copy link
Contributor

mmcky commented Jul 31, 2025

@jstac this is now running well.

I have just completed a review. Nice, quick introduction to JAX. The difference in generating random numbers is pretty interesting and a minor inconvenience when using jax (having to use keys) but would assist with reproducibility with the lectures :-)

There was only one issue re: timing. There are a couple of instances where you write about removing compile time and the total run time is higher (due to higher sys time). The actual user time is lower but this is confusing for new programmers.

Screenshot 2025-07-31 at 3 29 38 pm

and
Screenshot 2025-07-31 at 3 30 20 pm

Perhaps we should remove the second part and just focus on overall execution time?

@jstac
Copy link
Contributor Author

jstac commented Jul 31, 2025

Thanks @mmcky , nice catch. Please go ahead with your suggestion.

@github-actions github-actions bot temporarily deployed to pull request August 1, 2025 00:55 Inactive
@mmcky mmcky merged commit 9e0844e into main Aug 1, 2025
5 checks passed
@mmcky mmcky deleted the add_jax branch August 1, 2025 00:57
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.

2 participants