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

【Hackathon 8th No.14】CoNFiLD 论文复现 #1094

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

Conversation

ADream-ki
Copy link
Contributor

@ADream-ki ADream-ki commented Mar 5, 2025

PR types

New features

PR changes

APIs

Describe

【Hackathon 8th No.14】CoNFiLD 论文复现
PaddlePaddle/community#1080

Copy link

paddle-bot bot commented Mar 5, 2025

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Mar 5, 2025

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.
1 out of 2 committers have signed the CLA.

✅ ADream-ki
❌ ADream_ki


ADream_ki seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@HydrogenSulfate
Copy link
Collaborator

你好,麻烦签署一下cla

@ADream-ki
Copy link
Contributor Author

你好,麻烦签署一下cla

已经签署了,数据和权重文件还在转

@luotao1
Copy link
Collaborator

luotao1 commented Mar 5, 2025

请先提交 RFC 设计文档

@ADream-ki
Copy link
Contributor Author

请先提交 RFC 设计文档

已经加上了,请review

@ADream-ki
Copy link
Contributor Author

请问权重文件和推理所用的文件该如何上传? @HydrogenSulfate

@HydrogenSulfate
Copy link
Collaborator

请问权重文件和推理所用的文件该如何上传? @HydrogenSulfate

可以发送到[email protected],我上传后发你链接,你再更新到文档里

@ADream-ki
Copy link
Contributor Author

请问权重文件和推理所用的文件该如何上传? @HydrogenSulfate

可以发送到[email protected],我上传后发你链接,你再更新到文档里

权重文件已经发送过去了

Comment on lines 50 to 51
data_path: /home/xinyang/store/projects/nfdiff/algo/elbow/uvp.npy
coor_path: /home/xinyang/store/projects/nfdiff/algo/elbow/coor.npy
Copy link
Collaborator

Choose a reason for hiding this comment

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

建议使用相对路径,可以是./data/xxx.npy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

建议使用相对路径,可以是./data/xxx.npy

推理文件我通过邮箱发过去了,这里是还没替换

Copy link
Contributor Author

Choose a reason for hiding this comment

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

建议使用相对路径,可以是./data/xxx.npy

请问那些数据文件的链接可以提供一下吗? 我发到邮箱了的

from ppsci.utils import logger


class Normalizer(object):
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
class Normalizer(object):
class Normalizer:



class Normalizer(object):
def __init__(self, params=[], method="-11", dim=None):
Copy link
Collaborator

Choose a reason for hiding this comment

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

参数最好不要使用如[]、{}这种可变类型

Comment on lines 24 to 41
class Normalizer(object):
def __init__(self, params=[], method="-11", dim=None):
self.params = params
self.method = method
self.dim = dim

def fit_normalize(self, data):
raise NotImplementedError

def normalize(self, new_data):
raise NotImplementedError

def denormalize(self, new_data_norm):
raise NotImplementedError

def get_params(self):
raise NotImplementedError

Copy link
Collaborator

Choose a reason for hiding this comment

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

建议去掉Normalizer,直接合并到Normalizer_ts里代码会更简单

Copy link
Contributor Author

Choose a reason for hiding this comment

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

建议去掉Normalizer,直接合并到Normalizer_ts里代码会更简单

Done

Comment on lines 50 to 56
data_path: /home/xinyang/store/projects/nfdiff/algo/elbow/uvp.npy
coor_path: /home/xinyang/store/projects/nfdiff/algo/elbow/coor.npy

normalizer:
method: '-11'
dim: 0
batch_size: 4
Copy link
Collaborator

Choose a reason for hiding this comment

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

移动到INFER字段下面,并且batch_size是重复的字段

Copy link
Contributor Author

Choose a reason for hiding this comment

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

移动到INFER字段下面,并且batch_size是重复的字段

Done

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.

5 participants