Conversation
Summary of ChangesHello @avik-pal, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request modifies the continuous integration (CI) setup for building documentation. The primary goal is to shift the execution of documentation tutorials from GPU (CUDA) runners to CPU runners. This change is crucial for enabling external contributions to the documentation, as the previous setup involving secrets on GPU runners posed a barrier. By standardizing on CPU runners, the project aims to streamline the contribution process and foster broader community engagement. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the documentation build process to run all tutorials on CPU runners, which is a great change to facilitate external contributions. The changes to switch from 'CUDA' to 'CPU' devices and update the memory limit environment variable are correct. I've added a suggestion to improve the code formatting for the tutorial lists, which will enhance readability, especially since automatic formatting is disabled for this file.
1c7d392 to
9a3f482
Compare
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## main #1595 +/- ##
===========================================
- Coverage 82.12% 19.38% -62.75%
===========================================
Files 171 159 -12
Lines 7073 6834 -239
===========================================
- Hits 5809 1325 -4484
- Misses 1264 5509 +4245 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
handling secrets prevents external doc contributions