tensorflow / lucid

A collection of infrastructure and tools for research in neural network interpretability.
Apache License 2.0
4.65k stars 655 forks source link

[*.py] Rename "Arguments:" to "Args:" #271

Closed SamuelMarks closed 3 years ago

SamuelMarks commented 3 years ago

I've written custom parsers and emitters for everything from docstrings to classes and functions. However, I recently came across an issue with the TensorFlow codebase: inconsistent use of Args: and Arguments: in its docstrings. It is easy enough to extend my parsers to support both variants, however it looks like Arguments: is wrong anyway, as per:

Therefore, only Args: is valid. This PR replaces them throughout the codebase.

PS: For related PRs, see tensorflow/tensorflow/pull/45420

ludwigschubert commented 3 years ago

Thanks, Human-Samuel or Script-Samuel! :-)

SamuelMarks commented 3 years ago

You're welcome!

BTW: I did the first one by hand and the rest via fastmod, fd and hub 🤫