Style fix for optimizers #7839

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

chenmoneygithub merged 1 commit into stanfordnlp:main from chenmoneygithub:optimizer-style-2

Feb 22, 2025

Collaborator

chenmoneygithub commented Feb 22, 2025 •

edited

Loading

Many lint fixes for optimizers in order to enable style check (probably in 3 weeks):

line width should be less than 120
docstring should follow google style
inline the code if possible
other random stuff, like raise error instead of using assert.

Mostly automated by ruff.


          style fix for optimizers

044ceab

chenmoneygithub merged commit f3d7f26 into stanfordnlp:main

4 checks passed

chenmoneygithub deleted the optimizer-style-2 branch

April 25, 2025 21:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet