PaddleScience icon indicating copy to clipboard operation
PaddleScience copied to clipboard

[Example] Add battery-electrochemical-performance prediction model

Open huohuohuohuohuo123 opened this issue 1 year ago • 29 comments

PR types

PR changes

Describe

huohuohuohuohuo123 avatar Aug 10 '24 16:08 huohuohuohuohuo123

Thanks for your contribution!

paddle-bot[bot] avatar Aug 10 '24 16:08 paddle-bot[bot]

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 10 '24 16:08 CLAassistant

你好,感谢提交PR,但是代码要从submodule改为提交实际的代码

HydrogenSulfate avatar Aug 11 '24 05:08 HydrogenSulfate

@huohuohuohuohuo123 请与最新develop代码合并一下,否则八百多个文件没办法review,参考步骤:https://paddlescience-docs.readthedocs.io/zh-cn/latest/zh/development/#41

HydrogenSulfate avatar Aug 12 '24 10:08 HydrogenSulfate

好的重新提交了

huohuohuohuohuo123 avatar Aug 12 '24 10:08 huohuohuohuohuo123

@huohuohuohuohuo123 提交PR前请阅读开发指南,遵守基本的开发规范:https://paddlescience-docs.readthedocs.io/zh-cn/latest/zh/development/

HydrogenSulfate avatar Aug 13 '24 11:08 HydrogenSulfate

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.9 out of 10 committers have signed the CLA.✅ HydrogenSulfate✅ Yang-Changhui✅ zhiminzhang0830✅ KennyNH✅ MayYouBeProsperous✅ cxxly✅ huohuohuohuohuo123✅ lijialin03✅ KaiCHEN-HT❌ github-actions[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

@huohuohuohuohuo123 另外请签署一下CLA协议

HydrogenSulfate avatar Aug 13 '24 11:08 HydrogenSulfate

您好,仔细阅读了开发文档,请问具体哪里有什么问题

huohuohuohuohuo123 avatar Aug 14 '24 15:08 huohuohuohuohuo123

新增一个案例的代码可以参考 #964 , #962

HydrogenSulfate avatar Aug 14 '24 15:08 HydrogenSulfate

重新提交了

huohuohuohuohuo123 avatar Aug 21 '24 06:08 huohuohuohuohuo123

运行脚本这里辛苦转换成可执行py脚本文件吧,另外看到里面有很多tensorflow等其他框架代码,辛苦检查并修正一下。可以把测试结果和向光说明补充在在MLP_LI.md文档中。

leeleolay avatar Aug 26 '24 03:08 leeleolay

向光说明是什么意思?重新更新了,辛苦看一下还有哪里需要更改

huohuohuohuohuo123 avatar Aug 26 '24 15:08 huohuohuohuohuo123

向光说明是什么意思?重新更新了,辛苦看一下还有哪里需要更改

相关说明,打错字了不好意思。

leeleolay avatar Aug 29 '24 06:08 leeleolay

哦哦,麻烦看一下还有什么需要修改的地方

huohuohuohuohuo123 avatar Sep 02 '24 04:09 huohuohuohuohuo123

现在提交的文件和说明文档的部分有些不太完整,文档格式上辛苦再检查一下,做一下修正和完善

leeleolay avatar Sep 27 '24 08:09 leeleolay

您好,文档部分的格式进行了一些修改,辛苦看一下,哪里是否需要修改

huohuohuohuohuo123 avatar Sep 30 '24 05:09 huohuohuohuohuo123

文档的部分辛苦按照链接里的教程再修改一下吧 https://paddlescience-docs.readthedocs.io/zh-cn/latest/zh/development/#3

leeleolay avatar Nov 01 '24 06:11 leeleolay

您好,已经按照文档链接修改了,辛苦看下哪里还有问题

huohuohuohuohuo123 avatar Nov 10 '24 15:11 huohuohuohuohuo123

1.加载文档的时候,该案例为能正确显示,还存在一些bug: image 2.另外example里的文件里,我看到有数据集,辛苦整理到一个文件夹里; 3.另外辛苦修正完后再运行一下检查一些是否有运行过程中存在的一些bug

关于训练好的模型权重文件,已经上传到云上,您可以将这个权重文件的链接参考其他案例附到案例介绍的文档中 https://paddle-org.bj.bcebos.com/paddlescience/models/MLP_LI/MLP_LI_pretrained.pdparams

leeleolay avatar Nov 11 '24 02:11 leeleolay

您好,之前说的问题已经修改,辛苦看一下哪里还有问题

huohuohuohuohuo123 avatar Nov 11 '24 15:11 huohuohuohuohuo123

image 运行MLP_LI.py文件运行错误,如果需要安装第三方包,请在文档里表明要安装的依赖包,并在文档里注明训练命令和数据集。文件夹中的数据集的位置比较散乱,请做一下整理。关于result_out,这个文件夹的作用是什么?里面还有一些图片格式的文件,看起来是补充文档里的信息,如何只和文档有关,请移动到文档相关的位置。

leeleolay avatar Nov 12 '24 05:11 leeleolay

您好,已提交修改,辛苦看一下

huohuohuohuohuo123 avatar Dec 05 '24 09:12 huohuohuohuohuo123

有如下问题,辛苦确认一下: 1.精度问题:
按照文档运行python MLP_LI.p,测试结果的精度和文档里有差距 58c1bbc6180a76b34d3ed602d9deb35b 2.文件存储路径问题,有中文,数据集改为data比较好 95e68c9bd543f611022c223e546a17cf

leeleolay avatar Jan 02 '25 07:01 leeleolay

训练结果 我这边的训练结果和文档中是一致的,不知道是什么原因和你运行的结果和文档的结果有差距,其他问题已经更正,辛苦看一下

huohuohuohuohuo123 avatar Jan 03 '25 13:01 huohuohuohuohuo123

提供的MLP_LI.py文件里没有提供加载预训练模型的代码和测试预训练模型的相关代码,使用提供的代码进行训练得到的模型进行测试后的精度结果是我发出的图片里的数字。 image 屏幕打印消息V C E建议在文档中表明各个符号的含义

leeleolay avatar Jan 12 '25 13:01 leeleolay

您好,已经修改,辛苦看一下

huohuohuohuohuo123 avatar Feb 13 '25 07:02 huohuohuohuohuo123

您好,上述问题已修改,辛苦看一下

huohuohuohuohuo123 avatar Mar 13 '25 15:03 huohuohuohuohuo123

@huohuohuohuohuo123 你好,请按照如下步骤格式化一下你的代码, image

HydrogenSulfate avatar Mar 14 '25 07:03 HydrogenSulfate

B1F7F286601C52A92FC3D15EF06D5480 提交的代码里的这两个地方的都可以删掉,文档里有附加数据集链接,代码里的数据集可以删除 6621a9c88bd3b09f1180b54647483886 文档里这个下载模型权重的链接,多了两个arg“-O ./checkpoints/MLP_LI_pretrained.pdparams” 可以删掉

leeleolay avatar Mar 14 '25 07:03 leeleolay

您好,已经修改,麻烦您辛苦看一下

huohuohuohuohuo123 avatar Apr 25 '25 05:04 huohuohuohuohuo123