Skip to content
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.

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

@ganshaoduo check if this should be used #72

Closed
todo bot opened this issue Jul 1, 2021 · 4 comments
Closed

@ganshaoduo check if this should be used #72

todo bot opened this issue Jul 1, 2021 · 4 comments

Comments

@todo
Copy link

todo bot commented Jul 1, 2021

# weight_decay = group["weight_decay"] # TODO: @ganshaoduo check if this should be used
beta1, beta2 = group["betas"]
eps = group["eps"]
for param_id, param in enumerate(group["params"]):
state = self.state[param]


This issue was generated by todo based on a TODO comment in 96cb6fe when #24 was merged. cc @BaguaSys.
@todo todo bot added the todo 🗒️ label Jul 1, 2021
@close-issue-app
Copy link

This issue is closed because it does not meet our issue template. Please read it.

@todo todo bot reopened this Jul 1, 2021
@close-issue-app
Copy link

This issue is closed because it does not meet our issue template. Please read it.

@todo
Copy link
Author

todo bot commented Jul 1, 2021

This issue has been reopened because the TODO comment still exists in bagua/torch_api/algorithms/onebit_adam.py, as of 6f62a84.


If this was not intentional, just remove the comment from your code. You can also set the reopenClosed config if you don't want this to happen at all anymore.

@create-issue-branch
Copy link

NOBLES5E pushed a commit that referenced this issue Sep 15, 2021
Bumps [serde](https://github.com/serde-rs/serde) from 1.0.128 to 1.0.129.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.128...v1.0.129)

---
updated-dependencies:
- dependency-name: serde
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
NOBLES5E pushed a commit that referenced this issue Sep 15, 2021
Bumps [serde](https://github.com/serde-rs/serde) from 1.0.128 to 1.0.129.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.128...v1.0.129)

---
updated-dependencies:
- dependency-name: serde
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants