Skip to content

fix: repair notebook CI (dead model, missing API key, pyarrow type bug)#348

Merged
andreatgretel merged 2 commits intomainfrom
andreatgretel/fix/notebook-ci
Feb 23, 2026
Merged

fix: repair notebook CI (dead model, missing API key, pyarrow type bug)#348
andreatgretel merged 2 commits intomainfrom
andreatgretel/fix/notebook-ci

Conversation

@andreatgretel
Copy link
Contributor

Summary

Test plan

…sing API key

- Replace `meta/llama-4-scout-17b-16e-instruct` (no longer serving on
  build.nvidia.com) with `nvidia/nemotron-nano-12b-v2-vl` (project default)
  in tutorial notebook 4
- Add `OPENROUTER_API_KEY` to the `build-notebooks` workflow so notebooks
  5 and 6 (which use OpenRouter for image generation) can authenticate
- Regenerate colab notebooks to reflect the model change
When image columns are loaded from parquet with pyarrow backend,
list values are pyarrow ListScalars, not Python lists. The
isinstance(x, list) check fails, causing the whole ListScalar to be
treated as a single path string (producing filenames ending in
`png')]`). Use isinstance(x, str) instead to correctly handle any
iterable type.
@andreatgretel andreatgretel requested a review from a team as a code owner February 23, 2026 14:15
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 23, 2026

Greptile Summary

Fixes three CI failures in notebook workflows: replaces a dead vision model, adds missing API key authentication, and fixes a type-checking bug for pyarrow-loaded data.

Confidence Score: 5/5

  • Safe to merge - fixes critical CI failures with targeted, well-tested changes
  • All three fixes address real production issues with minimal scope. Model replacement fixes 404 errors, API key addition enables authentication, and type check fix handles pyarrow types correctly. CI workflow already passed validating the fixes.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/build-notebooks.yml Added OPENROUTER_API_KEY env var to fix authentication for tutorials 5/6
docs/notebook_source/4-providing-images-as-context.py Replaced dead model meta/llama-4-scout-17b-16e-instruct with nvidia/nemotron-nano-12b-v2-vl
docs/notebook_source/6-editing-images-with-image-context.py Fixed display_image bug to handle pyarrow ListScalar values from parquet files

Last reviewed commit: 103799b

@andreatgretel andreatgretel merged commit 4635846 into main Feb 23, 2026
49 checks 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