anyscale / academy

Ray tutorials from Anyscale
https://anyscale.com
Apache License 2.0
580 stars 195 forks source link

minor fixes #7

Closed ceteri closed 4 years ago

ceteri commented 4 years ago

Misc. notes:

FWIW, when running in JupyterLab the (image) link in 04-Linear-Upper-Confidence-Bound.ipynb doesn't work (i.e., hyperlinking to a file in a parent directory?) That may be a matter of JupyterLab security. I find there are different semantics for rendering notebooks in JupyterLab vs. GitHub, and your selected trade-off is good. In any case, the Boken plot for it runs fine so that covers the JupyterLab side.

NB: in the 06-Market-Example.ipynb notebook the MarketBandit.seed() never gets used in our example -- although it's probably better to leave it in. For example, a user might run a more extended example and use that to analyze the bandit behaviors.

ceteri commented 4 years ago

Version diffs shown in GitHub for 06-Market-Example.ipynb have a bunch of minutiae included from the Boken run and perhaps also my version of JupyterLab rendering differently (e.g., different CDNs)

The main change that I made was toward the end, a minor rewording of the paragraph A final note: when you checkpoint...