Skip to content

Conversation

@samjmolyneux
Copy link
Collaborator

  1. Added a test that runs all notebooks in the examples folder to ensure that they run without error. This also included the addition of nbmake to the test dependencies.

  2. We agreed not to use Jupytext for the time being, so I added an nbstripout hook to the pre-commit. This refuses to commit notebooks until all meta data has been stripped.

  3. Of course 1. and 2. can be removed if we later decide to use Jupytext.

  4. Added examples/two_normal.ipynb. Notebook builds up an example graph using two normal distributions. Notebook then attempts to solve by running sgd on $\lVert \nabla \mathcal{L} \rVert_2^2$. Notebook then demonstrates that the objective function can be easily changed and solves the problem using a custom objective function based up minimising KKT residuals.

  5. Added some ruff ignores rules to examples folder. I think we probably don't care about these things missing in examples.

Closes #103.

* LaTeX-ify introduction

* Tidy graph construction section

* LaTeX-ify solver sections

* Clarify edges that are being added

* Update examples/two_normal.ipynb

* Update .pre-commit-config.yaml

* Language for parameters has been decided

---------

Co-authored-by: willGraham01 <[email protected]>
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