Hi, I was just checking the repository here (great work btw!) and noticed that the command instruction in the root README - marimo run marimonotebook/lvof.py can be improved by using the new uv integration features in marimo which introduced the --sandbox argument in the edit and run commands. More about sandboxing here. uv (a python package manager - like pip) is a requirement for this feature.
I can raise a PR for the same if you'd like.
Also, for some reason, I can't get the notebook working locally (in the sense that the data is not showing up in the graphs); tried cloning your repository and following the commands, etc. It says Data is not available on the graphs. Any path related issues I should be aware of?
PS: I'm a marimo ambassador; who also contributes to the marimo spotlights repository. I will be logging in your marimo notebook as part of our collection (the repo serves as an archive to community-spotlights notebooks).
Hi, I was just checking the repository here (great work btw!) and noticed that the command instruction in the root README -
marimo run marimonotebook/lvof.pycan be improved by using the new uv integration features in marimo which introduced the--sandboxargument in theeditandruncommands. More about sandboxing here.uv(a python package manager - like pip) is a requirement for this feature.I can raise a PR for the same if you'd like.
Also, for some reason, I can't get the notebook working locally (in the sense that the data is not showing up in the graphs); tried cloning your repository and following the commands, etc. It says Data is not available on the graphs. Any path related issues I should be aware of?
PS: I'm a marimo ambassador; who also contributes to the marimo spotlights repository. I will be logging in your marimo notebook as part of our collection (the repo serves as an archive to community-spotlights notebooks).