Skip to content

ci: add per-release Colab demo refresh workflow#746

Open
aditya-neuraco wants to merge 1 commit into
mainfrom
feat/colab-demo-refresh-workflow
Open

ci: add per-release Colab demo refresh workflow#746
aditya-neuraco wants to merge 1 commit into
mainfrom
feat/colab-demo-refresh-workflow

Conversation

@aditya-neuraco

@aditya-neuraco aditya-neuraco commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Features

  • Add a manual "Update Google Colab" GitHub Actions workflow and a companion script (admin/update_google_colab.py). Run once per release, it retrains the demo policy against the latest released neuracore version, downloads the trained model, uploads it to the public Google Cloud Storage bucket, bumps the version pinned in the notebook, and opens a pull request with the change.

Bugfixes

  • None

Items

Related PRs

@aditya-neuraco aditya-neuraco self-assigned this Jul 1, 2026
@aditya-neuraco aditya-neuraco added the version:patch non-breaking bug fixes; no input/output changes (except defaults) label Jul 1, 2026
@aditya-neuraco aditya-neuraco force-pushed the feat/colab-demo-refresh-workflow branch from ea0082a to 20de51b Compare July 1, 2026 18:57
@aditya-neuraco aditya-neuraco changed the title feat: fix Colab notebook, migrate artifacts to GCS and add per-release refresh workflow fix: update colab notebook, migrate artifacts to GCS and add per-release refresh workflow Jul 1, 2026
Comment thread admin/update_google_colab.py
@aditya-neuraco aditya-neuraco force-pushed the feat/colab-demo-refresh-workflow branch from 20de51b to 50adbed Compare July 6, 2026 10:44
@aditya-neuraco aditya-neuraco changed the title fix: update colab notebook, migrate artifacts to GCS and add per-release refresh workflow ci: add per-release Colab demo refresh workflow Jul 6, 2026
@aditya-neuraco

Copy link
Copy Markdown
Contributor Author

@sdas-neuraco I have moved the Colab update to #762 to unblock it.

@aditya-neuraco aditya-neuraco force-pushed the feat/colab-demo-refresh-workflow branch from 50adbed to f617b1a Compare July 6, 2026 10:57
@aditya-neuraco aditya-neuraco force-pushed the feat/colab-demo-refresh-workflow branch from f617b1a to d5307b9 Compare July 6, 2026 13:05

@sdas-neuraco sdas-neuraco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM w/c Also pls speek with Muneeb when he is back/ add it to his release document that this must be run before release, so platform team are also aware of this existance and workflow

TERMINAL_STATES = {"COMPLETED", "FAILED", "CANCELLED", "ERROR"}

# Object path of the per-release model within the public bucket.
MODEL_OBJECT = "colab/bigym_pretrained_model.nc.zip"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe we should version this, because there is a gap between deployment and when this script is run, where there will be a missmatch. Also would aid in potential rollbacks incase of a bad deployment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good idea. It will catch the version mis-match early as well.

if status != "COMPLETED":
raise RuntimeError(f"Training job {job_id} ended with status {status}")
model_path = _download_model(job_id, get_current_org())
output.parent.mkdir(parents=True, exist_ok=True)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

after the model is trained we should do some rollouts to check it is actually a trained and functinal model. to be extra safe since this is imediate client facing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay, will add a that after the model finishes training.

@aditya-neuraco aditya-neuraco added version:none tests, docs, CI only; no functional changes and removed version:patch non-breaking bug fixes; no input/output changes (except defaults) labels Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:none tests, docs, CI only; no functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants