Skip to content

Stop tracking cached files and Jupyter notebook hashes in doc builds #425

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jan 31, 2025

Conversation

vzhurba01
Copy link
Collaborator

@vzhurba01 vzhurba01 commented Jan 29, 2025

close #351

  • .doctrees contain cached build files and don't impact html rendering. Store them outside of html build path.
  • overview.md has page settings that triggers Jupyter notebook execution. Since the settings have minimal impact on displayed page, remove them to avoid hash regeneration.

@vzhurba01 vzhurba01 self-assigned this Jan 29, 2025
Copy link
Contributor

copy-pr-bot bot commented Jan 29, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@vzhurba01 vzhurba01 marked this pull request as draft January 29, 2025 22:18
@vzhurba01
Copy link
Collaborator Author

/ok to test

This comment has been minimized.

@vzhurba01
Copy link
Collaborator Author

/ok to test

2 similar comments
@vzhurba01
Copy link
Collaborator Author

/ok to test

@vzhurba01
Copy link
Collaborator Author

/ok to test

@vzhurba01
Copy link
Collaborator Author

/ok to test

@vzhurba01 vzhurba01 changed the title 351 trigger Stop tracking cached files and Jupyter notebook hashes in doc builds Jan 30, 2025
@vzhurba01 vzhurba01 added P1 Medium priority - Should do CI/CD CI/CD infrastructure labels Jan 30, 2025
@vzhurba01 vzhurba01 added this to the cuda.core beta 3 milestone Jan 30, 2025
@vzhurba01 vzhurba01 marked this pull request as ready for review January 30, 2025 00:39
@vzhurba01 vzhurba01 added the blocked This task is currently blocked by other tasks label Jan 30, 2025
@vzhurba01
Copy link
Collaborator Author

Labeling as blocked on #424. It builds off of a commit to notebook execution.

@vzhurba01
Copy link
Collaborator Author

/ok to test

@vzhurba01 vzhurba01 removed the blocked This task is currently blocked by other tasks label Jan 30, 2025
Copy link
Member

Choose a reason for hiding this comment

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

Q: with #424, do we still need this change?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We do because Jupyter notebook still generates new hashes for the jupytext and kernelspec for every commit

@vzhurba01 vzhurba01 merged commit d824c5b into NVIDIA:main Jan 31, 2025
70 checks passed
Copy link

Doc Preview CI
Preview removed because the pull request was closed or merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD CI/CD infrastructure P1 Medium priority - Should do
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Doc build CI might update too frequently
2 participants