-
Notifications
You must be signed in to change notification settings - Fork 187
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
【hydra No.1】 Adapt preprocess transforms to multiple arguments and adapt VolterraIDE to hydra #572
Conversation
Thanks for your contribution! |
"""Get sampling points for integral. | ||
|
||
Args: | ||
in_ (Dict[str, np.ndarray]): Raw input dict. | ||
input (Dict[str, np.ndarray]): Raw input dict. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weight和label用不用也写一下
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已更正
"""Get sampling points for integral. | ||
|
||
Args: | ||
in_ (Dict[str, np.ndarray]): Raw input dict. | ||
input (Dict[str, np.ndarray]): Raw input dict. | ||
|
||
Returns: | ||
Dict[str, np.ndarray]: Input dict contained sampling points. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
还有Return
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已更正
@@ -69,21 +70,31 @@ def u_solution_func( | |||
return np.abs(1 - (out["x"] ** 2 + out["y"] ** 2)) ** (1 + ALPHA / 2) | |||
|
|||
# set input transform |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这样叫让人以为是model.register_input_transform()的参数(虽然看调用可以看出不是),是不是改叫input data transform之类的比较好?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已修改
examples/ide/volterra_ide.py
Outdated
kernel_func, | ||
func, | ||
) | ||
} | ||
|
||
# set constraint | ||
ITERS_PER_EPOCH = 1 | ||
# set input transform |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已修改
…#572) * adapt preprocess transforms to multiple arguments * update code * update docs * fix document * refine code in IDE and FPDE
PR types
Bug fixes
PR changes
APIs
Describe
#571
vision.Compose
只能接收、返回单个输入,因此修改预处理的 Compse 为自定义 Compose 适配多输入的情况。