Fixes for class 1

[klamike]

WIP

[klamike]

Note all interactive elements are broken in the HTML preview. I think the recent BonitoBook.jl (tries to?) fix this (though it seems you would have to write JS code which.. is probably not worth it)

[klamike]

┌ Warning: Cannot extract version for inventory from /home/runner/work/LearningToControlClass/LearningToControlClass/Project.toml: no such file
└ @ Documenter.HTMLWriter ~/.julia/packages/Documenter/eoWm2/src/html/write_inventory.jl:89
┌ Warning: Please set `inventory_version` in the `HTML()` options passed to `makedocs`.
└ @ Documenter.HTMLWriter ~/.julia/packages/Documenter/eoWm2/src/html/write_inventory.jl:98

is this a problem?

[klamike]

CI took almost an hour! Is that expected?

[andrewrosemberg]

┌ Warning: Cannot extract version for inventory from /home/runner/work/LearningToControlClass/LearningToControlClass/Project.toml: no such file
└ @ Documenter.HTMLWriter ~/.julia/packages/Documenter/eoWm2/src/html/write_inventory.jl:89
┌ Warning: Please set `inventory_version` in the `HTML()` options passed to `makedocs`.
└ @ Documenter.HTMLWriter ~/.julia/packages/Documenter/eoWm2/src/html/write_inventory.jl:98

is this a problem?

Not a problem. its just that Documenter expects there to be a package but this repo is not structred as a package.

[andrewrosemberg]

CI took almost an hour! Is that expected?

Not expected. It takes somewhere between 3 to 13 minutes for me (see https://github.com/LearningToOptimize/LearningToControlClass/actions/runs/16893071727). Perhaps it is the CI documenter verion change?

[klamike]

CI took almost an hour! Is that expected?

Not expected. It takes somewhere between 3 to 13 minutes for me (see https://github.com/LearningToOptimize/LearningToControlClass/actions/runs/16893071727). Perhaps it is the CI documenter verion change?

Ah yeah probably due to the 1.10.6 pin I tried to get rid of the warning. Trying out just removing the manifest file now, do you think it's important to commit them?

[andrewrosemberg]

Not expected. It takes somewhere between 3 to 13 minutes for me (see https://github.com/LearningToOptimize/LearningToControlClass/actions/runs/16893071727). Perhaps it is the CI documenter verion change?

Ah yeah probably due to the 1.10.6 pin I tried to get rid of the warning. Trying out just removing the manifest file now, do you think it's important to commit them?

I want to avoid any students having problems with versions, and this is the usual way people prepare workshops. Nevertheless, I am thinking that we can probably just make each person compile their own pluto when submitting a class. This way we the manifests and project tomls of each class won't influence the documenter anymore.

[andrewrosemberg]

Thank you!