Skip to content
New issue

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

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

Already on GitHub? # to your account

chore: rename package to follow PEP8 naming convention #20

Merged
merged 11 commits into from
Jul 4, 2022

Conversation

XuehaiPan
Copy link
Member

@XuehaiPan XuehaiPan commented Jul 4, 2022

Use snake_case for packages and modules. Ref https://peps.python.org/pep-0008/#package-and-module-names.

  • TorchOpt -> torchopt
  • TorchOpt/_src/accelerated_op/adam_op/AdamOp.py -> torchopt/_src/accelerated_op/adam_op/adam_op.py
  • TorchOpt/_src/Optimzer.py -> torchopt/_src/optimizer/*.py
  • TorchOpt/_src/MetaOptimzer.py -> torchopt/_src/optimizer/meta/*.py

@Benjamin-eecs Benjamin-eecs requested a review from JieRen98 July 4, 2022 07:12
@XuehaiPan XuehaiPan marked this pull request as draft July 4, 2022 10:15
@XuehaiPan XuehaiPan marked this pull request as ready for review July 4, 2022 11:30
@Benjamin-eecs Benjamin-eecs merged commit dceae50 into metaopt:main Jul 4, 2022
@XuehaiPan XuehaiPan deleted the pep8 branch July 7, 2022 08:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants