Skip to content

Conversation

Liyulingyue
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

#788

Copy link

paddle-bot bot commented Feb 27, 2024

Thanks for your contribution!

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. 标题需以【PPSCI Export&Infer No.】开头
  2. 补一下报名信息
  3. 补充 Inference 函数

pretrained_model_path: checkpoints/paddle_mrms_model

INFER:
pretrained_model_path: checkpoints/paddle_mrms_model
Copy link
Collaborator

Choose a reason for hiding this comment

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

INFER的预训练模型路径默认值和EVAL不同,需要填写官方提供的预训练模型,避免用户再手动下载再通过参数指定。改为:https://paddle-org.bj.bcebos.com/paddlescience/models/nowcastnet/nowcastnet_pretrained.pdparams

elif cfg.mode == "export":
export(cfg)
else:
raise ValueError(f"cfg.mode should in ['train', 'eval'], but got '{cfg.mode}'")
Copy link
Collaborator

Choose a reason for hiding this comment

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

error信息里加上export

Comment on lines +105 to +107
input_spec = [
{key: InputSpec([None, 1], "float32", name=key) for key in model.input_keys},
]
Copy link
Collaborator

Choose a reason for hiding this comment

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

该模型的形状不是[None, 1],应该是[None, 29, 512, 512, 2]

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Feb 29, 2024
@luotao1
Copy link
Collaborator

luotao1 commented Mar 13, 2024

@Liyulingyue 张师傅,别忘了这个PR哈

@HydrogenSulfate
Copy link
Collaborator

Closed for PR #895 is merged

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

Labels

contributor HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants