Skip to content

Conversation

@SurbhiJainUSC
Copy link
Collaborator

@SurbhiJainUSC SurbhiJainUSC commented Jan 8, 2026

Description

  • Run all post-training jupyter notebooks in Github CI tests only when github event is schedule or any ipynb file is changed in the PR
  • Update documentation to run jupyter notebooks on VS Code
  • Update/fix post-training notebooks

Tests

CI tests & VS code run

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch 3 times, most recently from 01ca326 to 4d5c50f Compare January 8, 2026 19:52
@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch 26 times, most recently from e7d7268 to f352157 Compare January 9, 2026 18:08
@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch 2 times, most recently from 6cdb451 to fea6511 Compare January 12, 2026 23:35
Copy link
Collaborator

@A9isha A9isha left a comment

Choose a reason for hiding this comment

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

Excellent work Surbhi - thank you!

@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch 2 times, most recently from 9dac9fc to 3a44179 Compare January 13, 2026 22:08
@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch from 3a44179 to ecdf7d2 Compare January 13, 2026 22:18
Copy link
Collaborator

@NuojCheng NuojCheng left a comment

Choose a reason for hiding this comment

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

Thank you for bringing the new test feature!

@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch from ecdf7d2 to a1a39db Compare January 16, 2026 00:10
@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch 2 times, most recently from 5f849d8 to b064355 Compare January 16, 2026 00:34
@SurbhiJainUSC SurbhiJainUSC force-pushed the jupyter_notebook_tests branch from b064355 to cdd3dd8 Compare January 16, 2026 00:35
@copybara-service copybara-service bot merged commit 6bcd411 into main Jan 16, 2026
28 of 29 checks passed
@copybara-service copybara-service bot deleted the jupyter_notebook_tests branch January 16, 2026 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants