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.

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

polish(pu): delete unused enable_fast_timestep argument #855

Merged
merged 6 commits into from
Jan 27, 2025
Merged

polish(pu): delete unused enable_fast_timestep argument #855

merged 6 commits into from
Jan 27, 2025

Conversation

puyuan1996
Copy link
Collaborator

@puyuan1996 puyuan1996 commented Jan 27, 2025 •
edited by PaParaZz1
Loading

Description

  • delete unused enable_fast_timestep argument

Related Issue

#851

TODO

Check List

  • merge the latest version source branch/repo, and resolve all the conflicts
  • pass style check
  • pass all the tests

@puyuan1996 puyuan1996 added the config Update config label Jan 27, 2025
PaParaZz1
PaParaZz1 approved these changes Jan 27, 2025
View reviewed changes
ding/model/template/q_learning.py Outdated Show resolved Hide resolved
ding/model/template/q_learning.py Outdated Show resolved Hide resolved
ding/model/template/q_learning.py Outdated Show resolved Hide resolved
ding/model/template/q_learning.py Outdated Show resolved Hide resolved
ding/model/template/q_learning.py Outdated Show resolved Hide resolved
@PaParaZz1 PaParaZz1 mentioned this pull request Jan 27, 2025
Open
@puyuan1996 puyuan1996 mentioned this pull request Jan 27, 2025
Closed
@PaParaZz1 PaParaZz1 merged commit 64efcb3 into opendilab:main Jan 27, 2025
12 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PaParaZz1 PaParaZz1 PaParaZz1 approved these changes

Assignees
No one assigned
Labels
config Update config
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@puyuan1996 @PaParaZz1