-
Notifications
You must be signed in to change notification settings - Fork 24.9k
[Docs] Convert to markdown: torch.compiler_transformations.rst, torch.compiler_troubleshooting_old.rst, torch.compiler_troubleshooting.rst, torch.compiler.config.rst, torch.compiler.rst #155120
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/155120
Note: Links to docs will display an error until the docs builds have been completed. ❌ 28 New Failures, 6 Cancelled Jobs, 4 Unrelated FailuresAs of commit 27df1a9 with merge base ca0c298 ( NEW FAILURES - The following jobs have failed:
CANCELLED JOBS - The following jobs were cancelled. Please retry:
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
UNSTABLE - The following jobs are marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Please only have .rst -> .md files that were converted using |
@svekars The first commit only renamed |
@svekars I realized GitHub WebUI cannot follow the history with
|
@svekars I also found an issue in this failure of the lint process. I have two possible suggestions.
cc @albanD Part of the log file.
Current: 2264+2303 lines
Proposed: 1381+1420 lines
|
That's right. Git doesn't actually store the history of files in its database. Every commit actually stores the entire current state of the workspace. Git then compares the state of the current commit against its parent commit to determine what was changed when it shows you a diff between 2 commits (this is also why you can compare 2 completely random commits in a git log and don't have to compare them in order). When you rename a file Git treats the new filename and old filename as different files referencing the same blob object in the git database. For very small files where we have to convert rst to md formatting, if more than 50% of the lines are changed then GitHub's WebUI will show it as a deleted and new file because git doesn't detect enough similar lines between the 2 blob objects so is not able to map them as similar objects anymore and since its not tracking history of filename changes its not able to tell its the same file. On the commandline we can adjust the threshold by using I've found Scott Chacon's book very useful in explaining how commits are actually made and stored in the database https://schacon.github.io/gitbook/1_the_git_object_model.html which helps explain why this happens. From Git's perspective whether or not you use "git mv" or just delete and add the file makes no difference if enough lines are similar git will detect that the file was renamed. Edit: You can test this like this.
Hope this helps explain what's happening here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try fixing broken targets like suggested to resolve the build errors.
@svekars Thank you for pointing them out. I addressed them and fixed the similar broken targets. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two very minor issues - otherwise LGTM
@svekars Thank you for fixing issues. |
@svekars Should I split this PR into multiple PRs to have fewer than 2000 lines of difference in each PR? |
@kiszk yes, please split into a smaller PR and link here |
@svekars I may misunderstand. The file
|
…orch.compiler.rst (#155348) Part of changes #155040 (parent PR #155120) Pull Request resolved: #155348 Approved by: https://github.com/svekars
You've got some conflicts here that need to be resolved manually. |
Part of changes #155040 (parent PR #155120) Pull Request resolved: #155351 Approved by: https://github.com/svekars Co-authored-by: Svetlana Karslioglu <svekars@meta.com>
This PR is not used for updating rst files now. This is just a parent PR for the four child PRs. |
closing this as this is handled through other PRs. |
….compiler.config.rst (#155347) Part of changes #155040 (parent PR #155120) Pull Request resolved: #155347 Approved by: https://github.com/svekars
Fixes #155040
cc @svekars @sekyondaMeta @AlannaBurke