Skip to content

[kalman] Update to JAX#611

Open
xuanguang-li wants to merge 9 commits into
mainfrom
update_kalman
Open

[kalman] Update to JAX#611
xuanguang-li wants to merge 9 commits into
mainfrom
update_kalman

Conversation

@xuanguang-li

Copy link
Copy Markdown
Contributor

This pull request makes updates to the kalman.md notebook, modernizing the codebase by switching from NumPy to JAX.

Key changes include:

  • Replaced all numpy (np) operations with jax.numpy (jnp) and updated imports accordingly.
  • Refactored the bivariate Gaussian plotting logic: replaced the custom bivariate_normal and gen_gaussian_plot_vals functions with a vectorized, JAX-based implementation using jax.vmap.
  • Standardized the use of random seeds in simulations for reproducible results.
  • Adding mystnb metadata blocks to each figure-generating code cell.
  • Fixed minor typos and improved the clarity.

Comment thread lectures/kalman.md Outdated
@github-actions

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-611--sunny-cactus-210e3e.netlify.app (5be7a1f)

📚 Changed Lecture Pages: back_prop, bayes_nonconj, kalman, lqcontrol, mle, qr_decomp

@github-actions

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-611--sunny-cactus-210e3e.netlify.app (b1629e8)

📚 Changed Lecture Pages: back_prop, bayes_nonconj, kalman, lqcontrol, mle, qr_decomp

@mmcky mmcky marked this pull request as ready for review September 17, 2025 22:14
Comment thread lectures/kalman.md Outdated
@mmcky mmcky removed the in-work label Sep 17, 2025
@mmcky

mmcky commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Thanks @xuanguang-li -- @HumphreyYang this is now ready for review.

@mmcky mmcky requested a review from HumphreyYang September 17, 2025 22:17
@github-actions

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-611--sunny-cactus-210e3e.netlify.app (533e30f)

📚 Changed Lecture Pages: back_prop, bayes_nonconj, kalman, lqcontrol, mle, qr_decomp

@jstac

jstac commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

🤖 Status note for a future session — from a maintainer investigation on 2026-07-08 into why open-PR previews 404. Context only, not instructions.

Netlify preview: https://pr-611--sunny-cactus-210e3e.netlify.app/ currently returns 404.

Why previews are down (repo-wide findings)

1. This branch is stale — 193 commits behind main. A preview build compiles the whole site from this branch. This branch's lectures/house_auction.md still has unpinned !pip install prettytable, which now breaks on a wcwidth incompatibility. main fixed this on 2026-06-28 by pinning prettytable<3.18 (#939). This alone fails any rebuild of this branch until it's updated to main.

2. The arviz failure was a red herring — do NOT pin arviz or rewrite plotting. A 2026-07-07 rebuild also failed in ar1_bayes/ar1_turningpts with an arviz_plots figsize ValueError. That was a transient bug in an intermediate arviz-plots 1.x release, already fixed in arviz 1.2.0. Verified locally on a clean latest-stack venv: the real az.plot_trace(trace) cell (pymc + numpyro InferenceData) runs green. The lectures use only 1.x-compatible arviz APIs (plot_trace, summary, from_numpyro, compare).

Recommended first step for this PR

Update this branch to main (merge or rebase — pulls in #939 plus ~193 other commits), then let CI rebuild. On today's latest libraries the site builds clean, so the preview should return. house_auction is the known blocker; updating also picks up other since-merged fixes — rebuild and address any remaining per-lecture failures. Verify with:

curl -sI https://pr-611--sunny-cactus-210e3e.netlify.app/kalman.html

This PR touches: kalman.md. Last CI build: success@2025-09-18. Branch: 193 commits behind main as of 2026-07-08.

@xuanguang-li

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved by merging origin/main into this branch and fixing the lectures/kalman.md conflicts. Commit: eeb3b74.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants