Skip to content

Fix invalid notebook JSON to satisfy nbformat schema#40

Merged
phuo-nv merged 1 commit into
mainfrom
features/launchable-mean-variance-and-cuopt-label
Apr 23, 2026
Merged

Fix invalid notebook JSON to satisfy nbformat schema#40
phuo-nv merged 1 commit into
mainfrom
features/launchable-mean-variance-and-cuopt-label

Conversation

@phuo-nv
Copy link
Copy Markdown
Collaborator

@phuo-nv phuo-nv commented Apr 23, 2026

GitHub was reporting "Invalid Notebook: 'metadata' is a required property" on several notebooks. nbformat.validate() identified outputs missing required fields:

  • execute_result / display_data outputs missing metadata
  • execute_result outputs missing execution_count
  • stream outputs missing name (defaulted to stdout)

Affected files

  • brev/launchable-content.ipynb
  • notebooks/efficient_frontier.ipynb
  • notebooks/launchable.ipynb
  • notebooks/mean_variance_basic.ipynb

Verification

All 9 notebooks in the repo now pass nbformat.validate().

GitHub was reporting "Invalid Notebook: 'metadata' is a required property"
on several notebooks. nbformat.validate() identified outputs missing
required fields:

- execute_result / display_data outputs missing `metadata`
- execute_result outputs missing `execution_count`
- stream outputs missing `name` (defaulted to "stdout")

Affected files:
- brev/launchable-content.ipynb
- notebooks/efficient_frontier.ipynb
- notebooks/launchable.ipynb
- notebooks/mean_variance_basic.ipynb

After the fix, all notebooks pass nbformat.validate().

Made-with: Cursor
@phuo-nv phuo-nv merged commit e8a365f into main Apr 23, 2026
5 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.

1 participant