-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
added KRNN and Sandwich models and their example results based on Alpha360 #1414
Conversation
updated the result of KRNN and Sandwich models based on Alpha360
@microsoft-github-policy-service agree [company="MSRA"] |
@microsoft-github-policy-service agree company="MSRA" |
Contributor License Agreement installed |
@microsoft-github-policy-service agree company="MSRA" |
return y | ||
|
||
|
||
class KRNNEncoderBase(nn.Module): |
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.
Why we have 2 same KRNNEncoderBase
in the code?
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.
changed
return hids | ||
|
||
|
||
class CNNKRNNEncoder(nn.Module): |
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.
CNNKRNNEncoder have two similar version as well.
Please merge them
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.
changed
Description
Motivation and Context
How Has This Been Tested?
pytest qlib/tests/test_all_pipeline.py
under upper directory ofqlib
.Screenshots of Test Results (if appropriate):
Types of changes