Skip to content

Updated notebooks with open in colab and default payloads#61

Open
WanjiruCate wants to merge 8 commits intomainfrom
gesa-workshop-notebooks
Open

Updated notebooks with open in colab and default payloads#61
WanjiruCate wants to merge 8 commits intomainfrom
gesa-workshop-notebooks

Conversation

@WanjiruCate
Copy link
Copy Markdown
Contributor

@WanjiruCate WanjiruCate commented Apr 16, 2026

Summary

Fix view on colab for notebooks
Add default payloads for lab4 burnscars
Add view on colab lab 1,2,3,4

Related Issue (optional)

How to test this PR?

Screenshots / Logs (optional)

Checklist

  • This PR targets the main branch
  • I have added or updated relevant docs.
  • I have not included any secrets or credentials.
  • Linting and formatting checks pass.

fMurugi
fMurugi previously approved these changes Apr 17, 2026
@WanjiruCate WanjiruCate requested a review from EugeneGene April 20, 2026 08:35
@EugeneGene
Copy link
Copy Markdown
Contributor

EugeneGene commented Apr 21, 2026

Using an existing Lima Local deployment with new worktree + bootstrap setup for testing.

One issue with notebooks in general...instead of putting "# List all..." and "# Select ..." in a single cell, separate:
image

Better yet, we may want to put a place holder to cue the user to use one of the displayed id's:
image

@EugeneGene
Copy link
Copy Markdown
Contributor

Does the config file (.geostudio_config_file) have to have a "." prefix? I typically use jupyterlab which hides the files with "." prefix. Consequently, the user may have to go back and forth to the command line instead of staying in the juypterlab interface.

@EugeneGene
Copy link
Copy Markdown
Contributor

In workshop/docs/notebooks/lab2-onboarding-examples.ipynb
Change:
2. Navigate to the Inference Lab:

  • Click on "Inference Lab" in the left sidebar
  • Or go directly to: https://localhost:4180/inference
    To:
  1. Navigate to the Inference Lab:
    • Click on "Inference Lab" in the left sidebar
    • Or go directly to: https://localhost:4180/inference#inference

@WanjiruCate
Copy link
Copy Markdown
Contributor Author

Using an existing Lima Local deployment with new worktree + bootstrap setup for testing.

One issue with notebooks in general...instead of putting "# List all..." and "# Select ..." in a single cell, separate: image

Better yet, we may want to put a place holder to cue the user to use one of the displayed id's: image

Updated the notebooks to use the placeholder

@WanjiruCate
Copy link
Copy Markdown
Contributor Author

@WanjiruCate
Copy link
Copy Markdown
Contributor Author

Does the config file (.geostudio_config_file) have to have a "." prefix? I typically use jupyterlab which hides the files with "." prefix. Consequently, the user may have to go back and forth to the command line instead of staying in the juypterlab interface.

I just tested without the "." prefix and it works. I am unsure of updating that in the notebook, I've added documentation to tell the user to use geostudio_config_file if on jupyterlab. It is possible to view hidden files on google colab.

@WanjiruCate WanjiruCate enabled auto-merge May 5, 2026 08:10
@WanjiruCate WanjiruCate requested a review from fMurugi May 5, 2026 08:23
@WanjiruCate WanjiruCate self-assigned this May 5, 2026
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