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

[Example] Add yinglong #975

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

Conversation

robotLiberator
Copy link

ppsci冲突太多,将简化的代码放到了jointContribution/yinglong下

Copy link

paddle-bot bot commented Aug 17, 2024

Thanks for your contribution!

@luotao1 luotao1 self-assigned this Aug 19, 2024

## Example Usage

### 1. Download the data and model weights
Copy link
Collaborator

Choose a reason for hiding this comment

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

此次PR下包含模型训练、推理的代码,不包含通过静态图模型推理的代码,因此在Usage这一部分建议按照类似以下的方式进行组织:

  1. model pretrain
    相关的预训练命令 xxx
  2. model finetune
    相关的微调命令 xxx
  3. model inference
    相关的推理命令 xxx

Copy link
Collaborator

Choose a reason for hiding this comment

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

此文件与模型训练推理无关,可以删除

# num_timestamps=NUM_TIMESTAMPS,
# prefix="wind",
# )
# }
Copy link
Collaborator

Choose a reason for hiding this comment

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

无用的注释可以删除

)
solver.eval()
# visualize prediction from pretrained_model_path
# solver.visualize()
Copy link
Collaborator

Choose a reason for hiding this comment

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

同上

# )
# PRETRAINED_MODEL_PATH = (
# f"../output/hrrr_time_embedding_merge_train_finetune_{NUM_TIMESTAMPS-1}/checkpoints/best_model"
# )
Copy link
Collaborator

Choose a reason for hiding this comment

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

同上

# metric=metric,
# name="Sup_Validator",
# )
# validator = {sup_validator.name: sup_validator}
Copy link
Collaborator

Choose a reason for hiding this comment

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

同上

Copy link
Collaborator

Choose a reason for hiding this comment

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

没有用到的代码可以直接删除,例如autodiff、geometry等;部分用到的,例如loss、metric等只用到了其中的某几个类,没有用到的类也可以直接删除。可以全局搜索一下。尽量保持此PR是yinglong代码的最小实现,无关的代码可以删除下。

Copy link
Collaborator

Choose a reason for hiding this comment

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

可以删除fourcastnet_hrrr文件夹,将其中的文件直接放到examples下即可

@robotLiberator
Copy link
Author

已经将冗余的代码和注释删除,并调整了文件目录

@zhiminzhang0830
Copy link
Collaborator

@robotLiberator 目前的代码训练、推理过程中好像没有用到地理数据?而之前release的代码中用到了这种数据, geo_data

pred_data = predictor.predict(input_data_0, time_stamps, nwp_data, geo_data)

@HydrogenSulfate HydrogenSulfate changed the title 应龙模型 [Example] Add yinglong Aug 21, 2024
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

4 participants