pytorch
6dc32a93 - [GHA] Remove new lines from PR_BODY too to appease batch env var copying (#80548)

Commit
2 years ago
[GHA] Remove new lines from PR_BODY too to appease batch env var copying (#80548) https://github.com/pytorch/pytorch/pull/80543 doesn't work in preventing batch from interpreting the multiline env vars. We will remove the lines from these env vars instead, since PR_BODY and COMMIT_MESSAGES are both used to determine what disabled tests to not skip. Test plan is using the following below and making sure tests still pass, which they do. Summary: previous versions of sparsity utils either allowed for a leading '.' for fqns, or would only allow for that. Per discussion with ao team about fqns don't have a leading '.' fqn of root module is '' these utilities have been updated to align with these definitions. module_to_fqn was changed to not generate a leading '.' and output '' for root module fqn_to_module was changed to output the root rather than None for path='' get_arg_info_from_tensor_fqn had explicit handling for a leading '.' that was removed. The previous implementation overwrote the tensor_fqn if it had a leading '.' which resulted in undesirable behavior of rewriting arguments provided by the user. Also refactored utils to be simpler and added comments, formatting and test Test Plan: python test/test_ao_sparsity.py python test/test_ao_sparsity.py TestSparsityUtilFunctions Reviewers: Subscribers: Tasks: Tags: Pull Request resolved: https://github.com/pytorch/pytorch/pull/80548 Approved by: https://github.com/mehtanirav
Author
Committer
Parents
Loading