pytorch
af05fbb8 - Linter to avoid csv merge conflicts (#111163)

Commit
1 year ago
Linter to avoid csv merge conflicts (#111163) This PR addresses the persistent issue of merge conflicts in the benchmarks/dynamo/ci_expected_accuracy/ directory, specifically those arising from frequently updated CSV files. Based on @malfet suggestion, the solution implemented adds three spaces between each line in the CSV files. This approach has proven effective in preventing merge conflicts, as evidenced in [D50239634](https://www.internalfb.com/intern/diff/D50239634/). Regardless of these changes the extra new lines should still allow the csvs to be ingested as normal. If you have access to the diff: Normally, modifying a line that is later altered in the stack results in a merge conflict during restacking. With this new spacing strategy, lines that are not modified further down the stack will not trigger merge conflicts, achieving our intended outcome. Pull Request resolved: https://github.com/pytorch/pytorch/pull/111163 Approved by: https://github.com/malfet, https://github.com/huydhn
Author
Committer
Parents
Loading