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

Humanoid control new #1042

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

ChenControl
Copy link

PR types

I following the below instruciton to try to finish the question
【hard】题目6(4-10分):参与快乐开源或黑客松等其他官方开源项目,收获双份收益。
#379
PaddlePaddle/Paddle#69471

I add a Humanoid control in example with both training and evaluation

PR changes

mujoco_control.md

Describe

HydrogenSulfate and others added 3 commits May 31, 2023 11:25
* update links README.md and index.md from latest to release/1.0(test=document_fix)

* update install_setup.md

* update docstrings
Copy link

paddle-bot bot commented Dec 15, 2024

Thanks for your contribution!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. 文档格式请参考https://paddlescience-docs.readthedocs.io/zh-cn/latest/zh/examples/allen_cahn/
  2. 权重文件可以给一个百度云盘或者可行的链接,我这边上传到百度的bce上

Comment on lines +1 to +2
defaults:
- _self_
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eval_with_no_grad: true

EVAL:
pretrained_model_path: outputs_HumanoidControl/2024-12-15/22-02-39/mode=train/checkpoints/latest.pdparams
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pretrained_model_path: outputs_HumanoidControl/2024-12-15/22-02-39/mode=train/checkpoints/latest.pdparams
pretrained_model_path: null

@@ -0,0 +1,441 @@
# Copyright (c) 2023 PaddlePaddle Authors. All Rights Reserved.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2023 --> 2024

@@ -0,0 +1,441 @@
# Copyright (c) 2023 PaddlePaddle Authors. All Rights Reserved.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2023 --> 2024

Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感谢提交代码。

  1. 建议基于develop分支开发代码,而不是release分支
  2. 请不要对无关文件进行修改。

)

def forward(self, x):
state = paddle.to_tensor(x["state"], dtype="float32")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

数据在输入到模型之前就需要保证是tensor,而不是在forward中再转换

Comment on lines +133 to +136
# Set random seed
ppsci.utils.misc.set_random_seed(cfg.seed)
logger.init_logger("ppsci", osp.join(cfg.output_dir, "train.log"), "info")

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yaml修改完之后,这几行可以删除

Comment on lines +187 to +191
"sampler": {
"name": "BatchSampler",
"drop_last": False,
"shuffle": False,
},
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里可以删除

Suggested change
"sampler": {
"name": "BatchSampler",
"drop_last": False,
"shuffle": False,
},

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

收到,谢谢这么多指点。修改后在提交!

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

Successfully merging this pull request may close these issues.

2 participants