EleutherAI / elk

Keeping language models honest by directly eliciting knowledge encoded in their activations.
MIT License
186 stars 33 forks source link

Replace flake with ruff #154

Closed thejaminator closed 1 year ago

thejaminator commented 1 year ago

Yep, ruff is pretty fast and nice

Still evaluating the docstrings feature I don't think enforcing rules "D" for docs everywhere is a good idea. There's still quite alot of false positives.

E.g. Ruff will complain about child override methods not having a docstring when the parent has it. https://github.com/charliermarsh/ruff/issues/970#issuecomment-1341901539

Closes https://github.com/EleutherAI/elk/issues/143