Skip to content

Codespace for tutorial based on Dockerfile #142

Merged
vissarion merged 18 commits intoGeomScale:developfrom
hariszaf:new_docker
Apr 26, 2026
Merged

Codespace for tutorial based on Dockerfile #142
vissarion merged 18 commits intoGeomScale:developfrom
hariszaf:new_docker

Conversation

@hariszaf
Copy link
Copy Markdown
Collaborator

In this PR I address:

This is done thanks to the devcontainer.json which enables firing codespaces based on a Dockerfile.

In our case, a Dockerfile is good to have to enable dingo running in non-Linux environments; this is why I also added a part on the main README on how to do so (see #124) .

Last, I edited the notebook tutorial (`dingo_tutorial.ipynb) to include cases for sampling with other methods than MMCS.

Copy link
Copy Markdown
Member

@vissarion vissarion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great addition! Thanks @hariszaf. So now both codespaces and colab run the same tutorial file? How do they synchronize?

Comment thread README.md Outdated
docker run --rm -it -v <path_to_WLS_license>:/opt/gurobi/gurobi.lic -v <path_to_your_model>:/data dingo
```

> **Remember** for this use need the WLS Gurobi license.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

phrasing here need some check.

@hariszaf
Copy link
Copy Markdown
Collaborator Author

In my understanding Google Colab is not needed anymore.

With just a click, in ~3 minutes, you get your codespace and you are good to go!

I cannot think of a reason for supporting both Colab and codespace.
If you do though, the best option to me would be to have just a chunk on top of the notebook, to be performed only in case the notebook is used with Colab -- not two notebooks.

But, still, to me this would not be necessary and would add some noise.

@vissarion
Copy link
Copy Markdown
Member

OK, I agree with keeping only one, then we should add a note on top of codelab after merging this. There is only a minor comment for you.

@vissarion vissarion merged commit c2e1dc0 into GeomScale:develop Apr 26, 2026
1 check passed
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.

2 participants