🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/969

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 9006929 with merge base c0a81f9 ():
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

So for linting feel free to add a new file or folder here and make all the changes you need https://github.com/pytorch/ao/blob/main/ruff.toml

It's tricky for us to merge something like this since eveyrone would need to rebase their code

So for linting feel free to add a new file or folder here and make all the changes you need https://github.com/pytorch/ao/blob/main/ruff.toml

It's tricky for us to merge something like this since eveyrone would need to rebase their code

thank you for the reference. that makes sense.
i then only updated the GitHub workflow to include the new rule. this renders no difference on the list of included files in the toml config.

Ok cool! Can you remove unused imports in files erroring in CI and we can merge this

Ok cool! Can you remove unused imports in files erroring in CI and we can merge this

turns out the ruff linter is configured to:

1. format w.r.t the listed files
2. apply the limited ruleset for all files

my change is to include a stricter linter rule but only applies to the allowlist, kind of between 1 and 2